commit bc4869293d7869c06e3f8c18241447ec5b701b58
author Lei Zhang <thestig@chromium.org> Mon Jan 29 15:30:46 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Mon Jan 29 15:30:46 2018 +0000
tree 00f8bbe2e40223dac30951ef0ae684228381f81b
parent 580f5dc88e3cbe8ed08661b829e2f560870e8b71

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>

fxjs/cjs_field.cpp

diff --git a/fxjs/cjs_field.cpp b/fxjs/cjs_field.cpp
index 0f06be0..7f530cf 100644
--- a/fxjs/cjs_field.cpp
+++ b/fxjs/cjs_field.cpp
@@ -2320,12 +2320,10 @@
   }
   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);