diff options
| author | Lei Zhang <thestig@chromium.org> | 2018-10-10 21:34:17 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-10-10 21:34:17 +0000 |
| commit | 7db136abc305630fb9ba9754a2c371c0e7ae8237 (patch) | |
| tree | b04ed80e31df6677881d1b0b8901d7669c14d7c2 /fpdfsdk/cpdfsdk_interform.cpp | |
| parent | e2210acfff622ac73b0783e1523615753484073b (diff) | |
| download | pdfium-7db136abc305630fb9ba9754a2c371c0e7ae8237.tar.xz | |
Remove more impossible checks in form filler code.
Also fix some nits and inline some calls.
Change-Id: Ie962f416b8ecd51ce05341e5511129ea0602fa8e
Reviewed-on: https://pdfium-review.googlesource.com/c/43604
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fpdfsdk/cpdfsdk_interform.cpp')
| -rw-r--r-- | fpdfsdk/cpdfsdk_interform.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fpdfsdk/cpdfsdk_interform.cpp b/fpdfsdk/cpdfsdk_interform.cpp index 2cc20f9d98..66a52b6dee 100644 --- a/fpdfsdk/cpdfsdk_interform.cpp +++ b/fpdfsdk/cpdfsdk_interform.cpp @@ -391,12 +391,11 @@ bool CPDFSDK_InterForm::OnKeyStrokeCommit(CPDF_FormField* pFormField, if (!action.GetDict()) return true; - CPDFSDK_ActionHandler* pActionHandler = m_pFormFillEnv->GetActionHandler(); CPDFSDK_FieldAction fa; fa.bModifier = false; fa.bShift = false; fa.sValue = csValue; - pActionHandler->DoAction_FieldJavaScript( + m_pFormFillEnv->GetActionHandler()->DoAction_FieldJavaScript( action, CPDF_AAction::KeyStroke, m_pFormFillEnv.Get(), pFormField, &fa); return fa.bRC; } @@ -411,12 +410,11 @@ bool CPDFSDK_InterForm::OnValidate(CPDF_FormField* pFormField, if (!action.GetDict()) return true; - CPDFSDK_ActionHandler* pActionHandler = m_pFormFillEnv->GetActionHandler(); CPDFSDK_FieldAction fa; fa.bModifier = false; fa.bShift = false; fa.sValue = csValue; - pActionHandler->DoAction_FieldJavaScript( + m_pFormFillEnv->GetActionHandler()->DoAction_FieldJavaScript( action, CPDF_AAction::Validate, m_pFormFillEnv.Get(), pFormField, &fa); return fa.bRC; } |