From bc4869293d7869c06e3f8c18241447ec5b701b58 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Mon, 29 Jan 2018 15:30:46 +0000 Subject: 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 Reviewed-by: dsinclair --- fxjs/cjs_field.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'fxjs') 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); -- cgit v1.2.3