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

Unified Diff: fpdfsdk/formfiller/cffl_checkbox.cpp

Issue 2357203003: Make the I in IFormFiller explicit (Closed)
Patch Set: Created 4 years, 3 months 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
Index: fpdfsdk/formfiller/cffl_checkbox.cpp
diff --git a/fpdfsdk/formfiller/cffl_checkbox.cpp b/fpdfsdk/formfiller/cffl_checkbox.cpp
index ef1bf18ea659151456a4ce80587f4cef2790e0f0..fdc5614002832a655c0f2d9ed32433db9bb2e509 100644
--- a/fpdfsdk/formfiller/cffl_checkbox.cpp
+++ b/fpdfsdk/formfiller/cffl_checkbox.cpp
@@ -42,8 +42,9 @@ FX_BOOL CFFL_CheckBox::OnChar(CPDFSDK_Annot* pAnnot,
switch (nChar) {
case FWL_VKEY_Return:
case FWL_VKEY_Space: {
- CFFL_IFormFiller* pIFormFiller = m_pEnv->GetIFormFiller();
- ASSERT(pIFormFiller);
+ CFFL_InteractiveFormFiller* pInteractiveFormFiller =
+ m_pEnv->GetInteractiveFormFiller();
+ ASSERT(pInteractiveFormFiller);
Tom Sepez 2016/09/21 18:15:30 nit: we'll segv at line 55, not sure this helps.
dsinclair 2016/09/21 18:34:01 Dropped the local all together.
CPDFSDK_PageView* pPageView = pAnnot->GetPageView();
ASSERT(pPageView);
@@ -51,7 +52,8 @@ FX_BOOL CFFL_CheckBox::OnChar(CPDFSDK_Annot* pAnnot,
FX_BOOL bReset = FALSE;
FX_BOOL bExit = FALSE;
- pIFormFiller->OnButtonUp(m_pWidget, pPageView, bReset, bExit, nFlags);
+ pInteractiveFormFiller->OnButtonUp(m_pWidget, pPageView, bReset, bExit,
+ nFlags);
if (bReset)
return TRUE;

Powered by Google App Engine
This is Rietveld 408576698