diff options
| author | Lei Zhang <thestig@chromium.org> | 2015-10-02 14:01:37 -0700 |
|---|---|---|
| committer | Lei Zhang <thestig@chromium.org> | 2015-10-02 14:01:37 -0700 |
| commit | ac67d4765a8ac36cd00c9fc8b6f2b80a3e1cff72 (patch) | |
| tree | c6c9718ccebfc035eeda548ae7f4f0cb9ef4059e /fpdfsdk/src/formfiller/FFL_IFormFiller.cpp | |
| parent | b8a074711108baf04017d86207e08c19ff327d8e (diff) | |
| download | pdfium-ac67d4765a8ac36cd00c9fc8b6f2b80a3e1cff72.tar.xz | |
Do not call into formfiller code with a NULL PageView.
BUG=537173
R=jun_fang@foxitsoftware.com, tsepez@chromium.org
Review URL: https://codereview.chromium.org/1376093002 .
Diffstat (limited to 'fpdfsdk/src/formfiller/FFL_IFormFiller.cpp')
| -rw-r--r-- | fpdfsdk/src/formfiller/FFL_IFormFiller.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp b/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp index f8f8c6a943..69ff93280e 100644 --- a/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp +++ b/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp @@ -459,7 +459,7 @@ FX_BOOL CFFL_IFormFiller::OnSetFocus(CPDFSDK_Annot* pAnnot, FX_UINT nFlag) { } if (CFFL_FormFiller* pFormFiller = GetFormFiller(pAnnot, TRUE)) - return pFormFiller->SetFocusForAnnot(pAnnot, nFlag); + pFormFiller->SetFocusForAnnot(pAnnot, nFlag); return TRUE; } @@ -470,8 +470,7 @@ FX_BOOL CFFL_IFormFiller::OnKillFocus(CPDFSDK_Annot* pAnnot, FX_UINT nFlag) { ASSERT(pAnnot->GetPDFAnnot()->GetSubType() == "Widget"); if (CFFL_FormFiller* pFormFiller = GetFormFiller(pAnnot, FALSE)) { - if (!pFormFiller->KillFocusForAnnot(pAnnot, nFlag)) - return FALSE; + pFormFiller->KillFocusForAnnot(pAnnot, nFlag); if (!m_bNotifying) { CPDFSDK_Widget* pWidget = (CPDFSDK_Widget*)pAnnot; |