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 --- core/fpdfdoc/cpdf_formfield.cpp | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) (limited to 'core') diff --git a/core/fpdfdoc/cpdf_formfield.cpp b/core/fpdfdoc/cpdf_formfield.cpp index df59369723..686054f9f1 100644 --- a/core/fpdfdoc/cpdf_formfield.cpp +++ b/core/fpdfdoc/cpdf_formfield.cpp @@ -168,14 +168,9 @@ bool CPDF_FormField::ResetField(bool bNotify) { int iCount = CountControls(); if (iCount) { // TODO(weili): Check whether anything special needs to be done for - // unison field. Otherwise, merge these branches. - if (IsUnison(this)) { - for (int i = 0; i < iCount; i++) - CheckControl(i, GetControl(i)->IsDefaultChecked(), false); - } else { - for (int i = 0; i < iCount; i++) - CheckControl(i, GetControl(i)->IsDefaultChecked(), false); - } + // unison field. (When IsUnison(this) returns true/false.) + for (int i = 0; i < iCount; i++) + CheckControl(i, GetControl(i)->IsDefaultChecked(), false); } if (bNotify && m_pForm->GetFormNotify()) m_pForm->GetFormNotify()->AfterCheckedStatusChange(this); -- cgit v1.2.3