diff options
| author | Lei Zhang <thestig@chromium.org> | 2018-01-29 15:30:46 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-29 15:30:46 +0000 |
| commit | bc4869293d7869c06e3f8c18241447ec5b701b58 (patch) | |
| tree | 00f8bbe2e40223dac30951ef0ae684228381f81b /fxjs/cjs_field.cpp | |
| parent | 580f5dc88e3cbe8ed08661b829e2f560870e8b71 (diff) | |
| download | pdfium-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/cjs_field.cpp')
| -rw-r--r-- | fxjs/cjs_field.cpp | 10 |
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); |