diff options
author | Tom Sepez <tsepez@chromium.org> | 2015-07-27 12:08:12 -0700 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2015-07-27 12:08:12 -0700 |
commit | 8d2aae7ee320da3a8ffe01c57e38b3f98443257d (patch) | |
tree | e694c6f82ec72fa46e6172b4475996b30d3f6a3a /fpdfsdk/src/pdfwindow/PWL_IconList.cpp | |
parent | ff46aaf499edcf153ee2f57c7016587aa96dcfa0 (diff) | |
download | pdfium-8d2aae7ee320da3a8ffe01c57e38b3f98443257d.tar.xz |
Revert "FX Bool considered harmful, part 3"
This reverts commit ff46aaf499edcf153ee2f57c7016587aa96dcfa0.
TBR=thestig@chromium.org
Review URL: https://codereview.chromium.org/1255293002 .
Diffstat (limited to 'fpdfsdk/src/pdfwindow/PWL_IconList.cpp')
-rw-r--r-- | fpdfsdk/src/pdfwindow/PWL_IconList.cpp | 60 |
1 files changed, 30 insertions, 30 deletions
diff --git a/fpdfsdk/src/pdfwindow/PWL_IconList.cpp b/fpdfsdk/src/pdfwindow/PWL_IconList.cpp index ecbb0fd3c0..157eba633d 100644 --- a/fpdfsdk/src/pdfwindow/PWL_IconList.cpp +++ b/fpdfsdk/src/pdfwindow/PWL_IconList.cpp @@ -22,7 +22,7 @@ CPWL_IconList_Item::CPWL_IconList_Item() : m_nIconIndex(-1), m_pData(NULL), - m_bSelected(false), + m_bSelected(FALSE), m_pText(NULL) { } @@ -67,7 +67,7 @@ void CPWL_IconList_Item::DrawThisAppearance(CFX_RenderDevice* pDevice, CPDF_Matr m_crIcon, m_pText->GetTextColor(), GetTransparency()); } -void CPWL_IconList_Item::SetSelect(bool bSelected) +void CPWL_IconList_Item::SetSelect(FX_BOOL bSelected) { m_bSelected = bSelected; @@ -78,7 +78,7 @@ void CPWL_IconList_Item::SetSelect(bool bSelected) } -bool CPWL_IconList_Item::IsSelected() const +FX_BOOL CPWL_IconList_Item::IsSelected() const { return m_bSelected; } @@ -121,7 +121,7 @@ void CPWL_IconList_Item::RePosChildWnd() rcClient.left += (PWL_IconList_ITEM_ICON_LEFTMARGIN + PWL_IconList_ITEM_WIDTH + PWL_IconList_ITEM_ICON_LEFTMARGIN); - m_pText->Move(rcClient, true, false); + m_pText->Move(rcClient, TRUE, FALSE); } void CPWL_IconList_Item::SetIconFillColor(const CPWL_Color& color) @@ -151,8 +151,8 @@ void CPWL_IconList_Item::OnDisabled() CPWL_IconList_Content::CPWL_IconList_Content(int32_t nListCount) : m_nSelectIndex(-1), m_pNotify(NULL), - m_bEnableNotify(true), - m_bMouseDown(false), + m_bEnableNotify(TRUE), + m_bMouseDown(FALSE), m_nListCount(nListCount) { } @@ -192,29 +192,29 @@ void CPWL_IconList_Content::CreateChildWnd(const PWL_CREATEPARAM & cp) } } -bool CPWL_IconList_Content::OnLButtonDown(const CPDF_Point & point, FX_DWORD nFlag) +FX_BOOL CPWL_IconList_Content::OnLButtonDown(const CPDF_Point & point, FX_DWORD nFlag) { SetFocus(); SetCapture(); - m_bMouseDown = true; + m_bMouseDown = TRUE; int32_t nItemIndex = FindItemIndex(point); SetSelect(nItemIndex); ScrollToItem(nItemIndex); - return true; + return TRUE; } -bool CPWL_IconList_Content::OnLButtonUp(const CPDF_Point & point, FX_DWORD nFlag) +FX_BOOL CPWL_IconList_Content::OnLButtonUp(const CPDF_Point & point, FX_DWORD nFlag) { - m_bMouseDown = false; + m_bMouseDown = FALSE; ReleaseCapture(); - return true; + return TRUE; } -bool CPWL_IconList_Content::OnMouseMove(const CPDF_Point & point, FX_DWORD nFlag) +FX_BOOL CPWL_IconList_Content::OnMouseMove(const CPDF_Point & point, FX_DWORD nFlag) { if (m_bMouseDown) { @@ -223,10 +223,10 @@ bool CPWL_IconList_Content::OnMouseMove(const CPDF_Point & point, FX_DWORD nFlag ScrollToItem(nItemIndex); } - return true; + return TRUE; } -bool CPWL_IconList_Content::OnKeyDown(FX_WORD nChar, FX_DWORD nFlag) +FX_BOOL CPWL_IconList_Content::OnKeyDown(FX_WORD nChar, FX_DWORD nFlag) { switch (nChar) { @@ -237,7 +237,7 @@ bool CPWL_IconList_Content::OnKeyDown(FX_WORD nChar, FX_DWORD nFlag) SetSelect(nItemIndex); ScrollToItem(nItemIndex); } - return true; + return TRUE; case FWL_VKEY_Down: if (m_nSelectIndex < m_nListCount-1) { @@ -245,10 +245,10 @@ bool CPWL_IconList_Content::OnKeyDown(FX_WORD nChar, FX_DWORD nFlag) SetSelect(nItemIndex); ScrollToItem(nItemIndex); } - return true; + return TRUE; } - return false; + return FALSE; } int32_t CPWL_IconList_Content::FindItemIndex(const CPDF_Point& point) @@ -307,8 +307,8 @@ void CPWL_IconList_Content::SetSelect(int32_t nIndex) { if (m_nSelectIndex != nIndex) { - SelectItem(m_nSelectIndex, false); - SelectItem(nIndex, true); + SelectItem(m_nSelectIndex, FALSE); + SelectItem(nIndex, TRUE); m_nSelectIndex = nIndex; if (IPWL_IconList_Notify* pNotify = GetNotify()) @@ -333,12 +333,12 @@ void CPWL_IconList_Content::SetNotify(IPWL_IconList_Notify* pNotify) m_pNotify = pNotify; } -void CPWL_IconList_Content::EnableNotify(bool bNotify) +void CPWL_IconList_Content::EnableNotify(FX_BOOL bNotify) { m_bEnableNotify = bNotify; } -void CPWL_IconList_Content::SelectItem(int32_t nItemIndex, bool bSelect) +void CPWL_IconList_Content::SelectItem(int32_t nItemIndex, FX_BOOL bSelect) { if (CPWL_IconList_Item* pItem = GetListItem(nItemIndex)) { @@ -423,7 +423,7 @@ void CPWL_IconList::RePosChildWnd() CPWL_Wnd::RePosChildWnd(); if (m_pListContent) - m_pListContent->Move(GetClientRect(), true, false); + m_pListContent->Move(GetClientRect(), TRUE, FALSE); } void CPWL_IconList::CreateChildWnd(const PWL_CREATEPARAM & cp) @@ -442,7 +442,7 @@ void CPWL_IconList::OnCreated() { pScrollBar->RemoveFlag(PWS_AUTOTRANSPARENT); pScrollBar->SetTransparency(255); - pScrollBar->SetNotifyForever(true); + pScrollBar->SetNotifyForever(TRUE); } } @@ -463,7 +463,7 @@ void CPWL_IconList::OnNotify(CPWL_Wnd* pWnd, FX_DWORD msg, intptr_t wParam, intp { if (!pScrollBar->IsVisible()) { - pScrollBar->SetVisible(true); + pScrollBar->SetVisible(TRUE); RePosChildWnd(); } else @@ -474,7 +474,7 @@ void CPWL_IconList::OnNotify(CPWL_Wnd* pWnd, FX_DWORD msg, intptr_t wParam, intp { if (pScrollBar->IsVisible()) { - pScrollBar->SetVisible(false); + pScrollBar->SetVisible(FALSE); RePosChildWnd(); } @@ -522,7 +522,7 @@ void CPWL_IconList::SetNotify(IPWL_IconList_Notify* pNotify) m_pListContent->SetNotify(pNotify); } -void CPWL_IconList::EnableNotify(bool bNotify) +void CPWL_IconList::EnableNotify(FX_BOOL bNotify) { m_pListContent->EnableNotify(bNotify); } @@ -552,7 +552,7 @@ void CPWL_IconList::SetIconFillColor(const CPWL_Color& color) m_pListContent->SetIconFillColor(color); } -bool CPWL_IconList::OnMouseWheel(short zDelta, const CPDF_Point & point, FX_DWORD nFlag) +FX_BOOL CPWL_IconList::OnMouseWheel(short zDelta, const CPDF_Point & point, FX_DWORD nFlag) { CPDF_Point ptScroll = m_pListContent->GetScrollPos(); CPDF_Rect rcScroll = m_pListContent->GetScrollArea(); @@ -583,10 +583,10 @@ bool CPWL_IconList::OnMouseWheel(short zDelta, const CPDF_Point & point, FX_DWOR if (CPWL_ScrollBar* pScrollBar = GetVScrollBar()) pScrollBar->OnNotify(this, PNM_SETSCROLLPOS, SBT_VSCROLL, (intptr_t)&ptNew.y); - return true; + return TRUE; } } - return false; + return FALSE; } |