From 7b1e53c0ef09da360304c6f4397082d09a945a7e Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 15 Sep 2017 15:43:11 -0700 Subject: Use unsigned types for app age, value age in widgets. Then if they roll over, it doesn't matter, since we only check for change. And then we can pull a silly check. Then remove some no-op calls where we didn't use the result. Change-Id: I35ba470b42fb8c32a6984999e0311b21729791ca Reviewed-on: https://pdfium-review.googlesource.com/14210 Commit-Queue: Tom Sepez Reviewed-by: Lei Zhang --- fpdfsdk/formfiller/cffl_formfiller.cpp | 10 +++---- fpdfsdk/formfiller/cffl_interactiveformfiller.cpp | 35 +++++++++++------------ fpdfsdk/formfiller/cffl_interactiveformfiller.h | 4 +-- 3 files changed, 23 insertions(+), 26 deletions(-) (limited to 'fpdfsdk/formfiller') diff --git a/fpdfsdk/formfiller/cffl_formfiller.cpp b/fpdfsdk/formfiller/cffl_formfiller.cpp index e59cd0520f..fd569c7201 100644 --- a/fpdfsdk/formfiller/cffl_formfiller.cpp +++ b/fpdfsdk/formfiller/cffl_formfiller.cpp @@ -361,8 +361,8 @@ CPWL_Wnd* CFFL_FormFiller::GetPDFWindow(CPDFSDK_PageView* pPageView, auto* pPrivateData = new CFFL_PrivateData; pPrivateData->pWidget = m_pWidget.Get(); pPrivateData->pPageView = pPageView; - pPrivateData->nWidgetAge = m_pWidget->GetAppearanceAge(); - pPrivateData->nValueAge = 0; + pPrivateData->nWidgetAppearanceAge = m_pWidget->GetAppearanceAge(); + pPrivateData->nWidgetValueAge = 0; cp.pAttachedData = pPrivateData; CPWL_Wnd* pNewWnd = NewPDFWindow(cp); m_Maps[pPageView] = pNewWnd; @@ -370,11 +370,11 @@ CPWL_Wnd* CFFL_FormFiller::GetPDFWindow(CPDFSDK_PageView* pPageView, } auto* pPrivateData = static_cast(pWnd->GetAttachedData()); - if (pPrivateData->nWidgetAge == m_pWidget->GetAppearanceAge()) + if (pPrivateData->nWidgetAppearanceAge == m_pWidget->GetAppearanceAge()) return pWnd; - return ResetPDFWindow(pPageView, - m_pWidget->GetValueAge() == pPrivateData->nValueAge); + return ResetPDFWindow( + pPageView, pPrivateData->nWidgetValueAge == m_pWidget->GetValueAge()); } void CFFL_FormFiller::DestroyPDFWindow(CPDFSDK_PageView* pPageView) { diff --git a/fpdfsdk/formfiller/cffl_interactiveformfiller.cpp b/fpdfsdk/formfiller/cffl_interactiveformfiller.cpp index 425c2add35..989539fad3 100644 --- a/fpdfsdk/formfiller/cffl_interactiveformfiller.cpp +++ b/fpdfsdk/formfiller/cffl_interactiveformfiller.cpp @@ -122,7 +122,7 @@ void CFFL_InteractiveFormFiller::OnMouseEnter( if (pWidget->GetAAction(CPDF_AAction::CursorEnter).GetDict()) { m_bNotifying = true; - int nValueAge = pWidget->GetValueAge(); + uint32_t nValueAge = pWidget->GetValueAge(); pWidget->ClearAppModified(); ASSERT(pPageView); @@ -154,9 +154,8 @@ void CFFL_InteractiveFormFiller::OnMouseExit(CPDFSDK_PageView* pPageView, CPDFSDK_Widget* pWidget = static_cast(pAnnot->Get()); if (pWidget->GetAAction(CPDF_AAction::CursorExit).GetDict()) { m_bNotifying = true; - pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nValueAge = pWidget->GetValueAge(); pWidget->ClearAppModified(); ASSERT(pPageView); @@ -191,9 +190,8 @@ bool CFFL_InteractiveFormFiller::OnLButtonDown( if (Annot_HitTest(pPageView, pAnnot->Get(), point) && pWidget->GetAAction(CPDF_AAction::ButtonDown).GetDict()) { m_bNotifying = true; - pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nValueAge = pWidget->GetValueAge(); pWidget->ClearAppModified(); ASSERT(pPageView); @@ -271,8 +269,8 @@ bool CFFL_InteractiveFormFiller::OnButtonUp(CPDFSDK_Annot::ObservedPtr* pAnnot, m_bNotifying = true; - int nAge = pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nAge = pWidget->GetAppearanceAge(); + uint32_t nValueAge = pWidget->GetValueAge(); ASSERT(pPageView); PDFSDK_FieldAction fa; @@ -376,9 +374,8 @@ bool CFFL_InteractiveFormFiller::OnSetFocus(CPDFSDK_Annot::ObservedPtr* pAnnot, CPDFSDK_Widget* pWidget = static_cast(pAnnot->Get()); if (pWidget->GetAAction(CPDF_AAction::GetFocus).GetDict()) { m_bNotifying = true; - pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nValueAge = pWidget->GetValueAge(); pWidget->ClearAppModified(); CFFL_FormFiller* pFormFiller = GetFormFiller(pWidget, true); @@ -718,8 +715,8 @@ bool CFFL_InteractiveFormFiller::OnClick(CPDFSDK_Annot::ObservedPtr* pAnnot, return false; m_bNotifying = true; - int nAge = pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nAge = pWidget->GetAppearanceAge(); + uint32_t nValueAge = pWidget->GetValueAge(); PDFSDK_FieldAction fa; fa.bModifier = CPDFSDK_FormFillEnvironment::IsCTRLKeyDown(nFlag); @@ -748,8 +745,8 @@ bool CFFL_InteractiveFormFiller::OnFull(CPDFSDK_Annot::ObservedPtr* pAnnot, return false; m_bNotifying = true; - int nAge = pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nAge = pWidget->GetAppearanceAge(); + uint32_t nValueAge = pWidget->GetValueAge(); PDFSDK_FieldAction fa; fa.bModifier = CPDFSDK_FormFillEnvironment::IsCTRLKeyDown(nFlag); @@ -801,8 +798,8 @@ bool CFFL_InteractiveFormFiller::OnPreOpen(CPDFSDK_Annot::ObservedPtr* pAnnot, return false; m_bNotifying = true; - int nAge = pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nAge = pWidget->GetAppearanceAge(); + uint32_t nValueAge = pWidget->GetValueAge(); PDFSDK_FieldAction fa; fa.bModifier = CPDFSDK_FormFillEnvironment::IsCTRLKeyDown(nFlag); @@ -832,8 +829,8 @@ bool CFFL_InteractiveFormFiller::OnPostOpen(CPDFSDK_Annot::ObservedPtr* pAnnot, return false; m_bNotifying = true; - int nAge = pWidget->GetAppearanceAge(); - int nValueAge = pWidget->GetValueAge(); + uint32_t nAge = pWidget->GetAppearanceAge(); + uint32_t nValueAge = pWidget->GetValueAge(); PDFSDK_FieldAction fa; fa.bModifier = CPDFSDK_FormFillEnvironment::IsCTRLKeyDown(nFlag); @@ -888,8 +885,8 @@ std::pair CFFL_InteractiveFormFiller::OnBeforeKeyStroke( CFX_AutoRestorer restorer(&m_bNotifying); m_bNotifying = true; - int nAge = privateData.pWidget->GetAppearanceAge(); - int nValueAge = privateData.pWidget->GetValueAge(); + uint32_t nAge = privateData.pWidget->GetAppearanceAge(); + uint32_t nValueAge = privateData.pWidget->GetValueAge(); CPDFSDK_FormFillEnvironment* pFormFillEnv = privateData.pPageView->GetFormFillEnv(); diff --git a/fpdfsdk/formfiller/cffl_interactiveformfiller.h b/fpdfsdk/formfiller/cffl_interactiveformfiller.h index ddab2adcda..0982f1d92b 100644 --- a/fpdfsdk/formfiller/cffl_interactiveformfiller.h +++ b/fpdfsdk/formfiller/cffl_interactiveformfiller.h @@ -157,8 +157,8 @@ class CFFL_PrivateData : public CPWL_Wnd::PrivateData { public: CPDFSDK_Widget* pWidget; CPDFSDK_PageView* pPageView; - int nWidgetAge; - int nValueAge; + uint32_t nWidgetAppearanceAge; + uint32_t nWidgetValueAge; }; #endif // FPDFSDK_FORMFILLER_CFFL_INTERACTIVEFORMFILLER_H_ -- cgit v1.2.3