summaryrefslogtreecommitdiff
path: root/fxjs
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2018-01-29 15:30:46 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-01-29 15:30:46 +0000
commitbc4869293d7869c06e3f8c18241447ec5b701b58 (patch)
tree00f8bbe2e40223dac30951ef0ae684228381f81b /fxjs
parent580f5dc88e3cbe8ed08661b829e2f560870e8b71 (diff)
downloadpdfium-bc4869293d7869c06e3f8c18241447ec5b701b58.tar.xz
Fix identical if/else blocks.
The CPWL_EditImpl::Backspace() instance occurred in commit 671a799. The other two instances where known cases. BUG=chromium:805881 Change-Id: Iae7cb8534e85342069d8753ced88543dad751e92 Reviewed-on: https://pdfium-review.googlesource.com/24050 Commit-Queue: dsinclair <dsinclair@chromium.org> Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs')
-rw-r--r--fxjs/cjs_field.cpp10
1 files changed, 4 insertions, 6 deletions
diff --git a/fxjs/cjs_field.cpp b/fxjs/cjs_field.cpp
index 0f06be0091..7f530cf4d8 100644
--- a/fxjs/cjs_field.cpp
+++ b/fxjs/cjs_field.cpp
@@ -2320,12 +2320,10 @@ CJS_Return Field::checkThisBox(
}
if (nWidget < 0 || nWidget >= pFormField->CountControls())
return CJS_Return(false);
- // TODO(weili): Check whether anything special needed for radio button,
- // otherwise merge these branches.
- if (pFormField->GetFieldType() == FormFieldType::kRadioButton)
- pFormField->CheckControl(nWidget, bCheckit, true);
- else
- pFormField->CheckControl(nWidget, bCheckit, true);
+
+ // TODO(weili): Check whether anything special needed for radio button.
+ // (When pFormField->GetFieldType() == FormFieldType::kRadioButton.)
+ pFormField->CheckControl(nWidget, bCheckit, true);
UpdateFormField(m_pFormFillEnv.Get(), pFormField, true, true, true);
return CJS_Return(true);