diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-11 16:38:31 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-11 16:38:31 +0000 |
| commit | c7f4322f1ac8ee1604ac3e24860530994575c9af (patch) | |
| tree | b0d90ac2bc06c293a6c15052141b32c19a283702 /xfa/fxfa/cxfa_ffwidgethandler.cpp | |
| parent | ce7df754b0608887a923d5083f0bbb999a735fce (diff) | |
| download | pdfium-c7f4322f1ac8ee1604ac3e24860530994575c9af.tar.xz | |
Rename CXFA_Calculate methods to make return clearer
This CL renames CXFA_Calculate::GetScript to make it clearer it can
return nullptr.
Change-Id: I1f5c19c16ea6cb1b3e7016ddac3f7f7fd6aa0320
Reviewed-on: https://pdfium-review.googlesource.com/22741
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffwidgethandler.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffwidgethandler.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/cxfa_ffwidgethandler.cpp b/xfa/fxfa/cxfa_ffwidgethandler.cpp index d4ead07ba9..87cef5fae4 100644 --- a/xfa/fxfa/cxfa_ffwidgethandler.cpp +++ b/xfa/fxfa/cxfa_ffwidgethandler.cpp @@ -195,7 +195,7 @@ bool CXFA_FFWidgetHandler::HasEvent(CXFA_WidgetAcc* pWidgetAcc, switch (eEventType) { case XFA_EVENT_Calculate: { CXFA_Calculate* calc = node->GetCalculateIfExists(); - return calc && calc->GetScript(); + return calc && calc->GetScriptIfExists(); } case XFA_EVENT_Validate: { CXFA_Validate* validate = node->GetValidateIfExists(); @@ -234,7 +234,7 @@ int32_t CXFA_FFWidgetHandler::ProcessEvent(CXFA_WidgetAcc* pWidgetAcc, return XFA_EVENTERROR_NotExist; if (node->IsUserInteractive()) return XFA_EVENTERROR_Disabled; - return node->ExecuteScript(m_pDocView, calc->GetScript(), pParam); + return node->ExecuteScript(m_pDocView, calc->GetScriptIfExists(), pParam); } default: break; |