Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(30)

Unified Diff: fpdfsdk/src/pdfwindow/PWL_Note.cpp

Issue 1547843003: Merge to XFA: Cleanup: Remove some dead code in CPWL_Note. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@xfa
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « fpdfsdk/include/pdfwindow/PWL_Note.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: fpdfsdk/src/pdfwindow/PWL_Note.cpp
diff --git a/fpdfsdk/src/pdfwindow/PWL_Note.cpp b/fpdfsdk/src/pdfwindow/PWL_Note.cpp
index d87c3df916c850ba742119e81e9f6f94acde6b1f..4ac237df7f732ef41830fc71dd6228c2da4e9c2b 100644
--- a/fpdfsdk/src/pdfwindow/PWL_Note.cpp
+++ b/fpdfsdk/src/pdfwindow/PWL_Note.cpp
@@ -1059,9 +1059,7 @@ CPWL_Note::CPWL_Note(IPopup_Note* pPopupNote,
m_pOptions(NULL),
m_pNoteNotify(pNoteNotify),
m_bResizing(FALSE),
- m_rcCaption(0, 0, 0, 0),
- m_bEnalbleNotify(TRUE),
- m_pPopupNote(pPopupNote) {}
+ m_bEnableNotify(TRUE) {}
CPWL_Note::~CPWL_Note() {}
@@ -1070,7 +1068,7 @@ IPWL_NoteItem* CPWL_Note::Reply() {
}
void CPWL_Note::EnableNotify(FX_BOOL bEnabled) {
- m_bEnalbleNotify = bEnabled;
+ m_bEnableNotify = bEnabled;
}
void CPWL_Note::RePosChildWnd() {
@@ -1234,46 +1232,11 @@ void CPWL_Note::RePosNoteChildren() {
rcContentsBar.left = rcContentsBar.right - PWL_SCROLLBAR_WIDTH;
rcContentsBar.Normalize();
m_pContentsBar->Move(rcContentsBar, TRUE, FALSE);
-
- m_rcCaption = rcClient;
- m_rcCaption.bottom = rcContents.top;
}
m_bResizing = FALSE;
}
-// TODO(thestig): Make this return an enum.
-// 0-normal / 1-caption / 2-leftbottom corner / 3-rightbottom corner / 4-close /
-// 5-options
-int32_t CPWL_Note::NoteHitTest(const CPDF_Point& point) const {
- GetClientRect();
-
- if (m_pSubject->WndHitTest(m_pSubject->ParentToChild(point)))
- return 1;
- if (m_pDateTime->WndHitTest(m_pDateTime->ParentToChild(point)))
- return 1;
- if (m_pAuthor->WndHitTest(m_pAuthor->ParentToChild(point)))
- return 1;
- if (m_pIcon->WndHitTest(m_pIcon->ParentToChild(point)))
- return 1;
-
- if (m_pContents->WndHitTest(m_pContents->ParentToChild(point)))
- return 0;
- if (m_pContentsBar->WndHitTest(m_pContentsBar->ParentToChild(point)))
- return 0;
-
- if (m_pCloseBox->WndHitTest(m_pCloseBox->ParentToChild(point)))
- return 4;
- if (m_pLBBox->WndHitTest(m_pLBBox->ParentToChild(point)))
- return 2;
- if (m_pRBBox->WndHitTest(m_pRBBox->ParentToChild(point)))
- return 3;
- if (m_pOptions->WndHitTest(m_pOptions->ParentToChild(point)))
- return 5;
-
- return 1;
-}
-
void CPWL_Note::CreateChildWnd(const PWL_CREATEPARAM& cp) {
CPWL_NoteItem::CreateChildWnd(cp);
@@ -1551,10 +1514,7 @@ const CPWL_Note* CPWL_Note::GetNote() const {
}
IPWL_NoteNotify* CPWL_Note::GetNoteNotify() const {
- if (m_bEnalbleNotify)
- return m_pNoteNotify;
-
- return NULL;
+ return m_bEnableNotify ? m_pNoteNotify : nullptr;
}
void CPWL_Note::SetIconType(int32_t nType) {
« no previous file with comments | « fpdfsdk/include/pdfwindow/PWL_Note.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698