summaryrefslogtreecommitdiff
path: root/fpdfsdk/formfiller/cffl_radiobutton.cpp
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-07-06 11:35:04 -0400
committerChromium commit bot <commit-bot@chromium.org>2017-07-06 19:12:07 +0000
commitfecd7506f715e7d6908445d4913015ce81c9b23d (patch)
tree5135b373c74d58409f919d73a523582592a1b7bb /fpdfsdk/formfiller/cffl_radiobutton.cpp
parent5fe9808f03dc8164161d5b421f650af1d8547f53 (diff)
downloadpdfium-fecd7506f715e7d6908445d4913015ce81c9b23d.tar.xz
Convert OnButtonUp to not have in-out params
This CL removes the in-out param from OnButtonUp in favour of a single bool return. Change-Id: I2d91466677adcb366ed1c2d0721ce5e2949ed074 Reviewed-on: https://pdfium-review.googlesource.com/7333 Reviewed-by: Henrique Nakashima <hnakashima@chromium.org> Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fpdfsdk/formfiller/cffl_radiobutton.cpp')
-rw-r--r--fpdfsdk/formfiller/cffl_radiobutton.cpp9
1 files changed, 4 insertions, 5 deletions
diff --git a/fpdfsdk/formfiller/cffl_radiobutton.cpp b/fpdfsdk/formfiller/cffl_radiobutton.cpp
index 9ffdf494ca..e36e1f7554 100644
--- a/fpdfsdk/formfiller/cffl_radiobutton.cpp
+++ b/fpdfsdk/formfiller/cffl_radiobutton.cpp
@@ -48,13 +48,12 @@ bool CFFL_RadioButton::OnChar(CPDFSDK_Annot* pAnnot,
CPDFSDK_PageView* pPageView = pAnnot->GetPageView();
ASSERT(pPageView);
- bool bReset = false;
- bool bExit = false;
CPDFSDK_Annot::ObservedPtr pObserved(m_pWidget.Get());
- m_pFormFillEnv->GetInteractiveFormFiller()->OnButtonUp(
- &pObserved, pPageView, bReset, bExit, nFlags);
- if (!pObserved || bReset || bExit)
+ if (m_pFormFillEnv->GetInteractiveFormFiller()->OnButtonUp(
+ &pObserved, pPageView, nFlags) ||
+ !pObserved) {
return true;
+ }
CFFL_FormFiller::OnChar(pAnnot, nChar, nFlags);
CPWL_RadioButton* pWnd = GetRadioButton(pPageView, true);