Clean up: remove unnecessary if-branches on initialization of the a11y prefs.

chrome/browser/chromeos/preferences.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/preferences.h"
 
 #include "base/chromeos/chromeos_version.h"
 #include "base/command_line.h"
 #include "base/i18n/time_formatting.h"
 #include "base/metrics/histogram.h"
@@ skipping 75 matching lines @@
       PrefServiceSyncable::SYNCABLE_PREF);
   prefs->RegisterBooleanPref(prefs::kPrimaryMouseButtonRight,
                              false,
                              PrefServiceSyncable::SYNCABLE_PREF);
   prefs->RegisterBooleanPref(prefs::kLabsMediaplayerEnabled,
                              false,
                              PrefServiceSyncable::UNSYNCABLE_PREF);
   prefs->RegisterBooleanPref(prefs::kLabsAdvancedFilesystemEnabled,
                              false,
                              PrefServiceSyncable::UNSYNCABLE_PREF);
-  // Check if the accessibility pref is already registered, which can happen
+  // Check if the accessibility prefs are already registered, which can happen
   // in WizardController::RegisterPrefs. We still want to try to register
-  // the pref here in case of Chrome/Linux with ChromeOS=1.
+  // the prefs here in case of Chrome/Linux with ChromeOS=1.
   if (prefs->FindPreference(prefs::kSpokenFeedbackEnabled) == NULL) {
     prefs->RegisterBooleanPref(prefs::kSpokenFeedbackEnabled,
                                false,
                                PrefServiceSyncable::UNSYNCABLE_PREF);
   }
   if (prefs->FindPreference(prefs::kHighContrastEnabled) == NULL) {
     prefs->RegisterBooleanPref(prefs::kHighContrastEnabled,
                                false,
                                PrefServiceSyncable::UNSYNCABLE_PREF);
   }
-  if (prefs->FindPreference(prefs::kScreenMagnifierEnabled) == NULL) {
-    prefs->RegisterBooleanPref(prefs::kScreenMagnifierEnabled,
-                               false,
-                               PrefServiceSyncable::SYNCABLE_PREF);
-  }
-  if (prefs->FindPreference(prefs::kScreenMagnifierScale) == NULL) {
-    prefs->RegisterDoublePref(prefs::kScreenMagnifierScale,
-                              std::numeric_limits<double>::min(),
-                              PrefServiceSyncable::UNSYNCABLE_PREF);
-  }
-  if (prefs->FindPreference(prefs::kShouldAlwaysShowAccessibilityMenu) ==
-      NULL) {
-    prefs->RegisterBooleanPref(prefs::kShouldAlwaysShowAccessibilityMenu,
-                               false,
-                               PrefServiceSyncable::UNSYNCABLE_PREF);
-  }
   if (prefs->FindPreference(prefs::kVirtualKeyboardEnabled) == NULL) {
     prefs->RegisterBooleanPref(prefs::kVirtualKeyboardEnabled,
                                false,
                                PrefServiceSyncable::UNSYNCABLE_PREF);
   }
+  prefs->RegisterBooleanPref(prefs::kScreenMagnifierEnabled,
+                             false,
+                             PrefServiceSyncable::SYNCABLE_PREF);
+  prefs->RegisterDoublePref(prefs::kScreenMagnifierScale,
+                            std::numeric_limits<double>::min(),
+                            PrefServiceSyncable::UNSYNCABLE_PREF);
+  prefs->RegisterBooleanPref(prefs::kShouldAlwaysShowAccessibilityMenu,
+                             false,
+                             PrefServiceSyncable::UNSYNCABLE_PREF);
   prefs->RegisterIntegerPref(prefs::kMouseSensitivity,
                              3,
                              PrefServiceSyncable::SYNCABLE_PREF);
   prefs->RegisterIntegerPref(prefs::kTouchpadSensitivity,
                              3,
                              PrefServiceSyncable::SYNCABLE_PREF);
   prefs->RegisterBooleanPref(prefs::kUse24HourClock,
                              base::GetHourClockType() == base::k24HourClock,
                              PrefServiceSyncable::SYNCABLE_PREF);
   prefs->RegisterBooleanPref(prefs::kDisableDrive,
@@ skipping 644 matching lines @@
 
   input_method::AutoRepeatRate rate;
   rate.initial_delay_in_ms = xkb_auto_repeat_delay_pref_.GetValue();
   rate.repeat_interval_in_ms = xkb_auto_repeat_interval_pref_.GetValue();
   DCHECK(rate.initial_delay_in_ms > 0);
   DCHECK(rate.repeat_interval_in_ms > 0);
   input_method::XKeyboard::SetAutoRepeatRate(rate);
 }
 
 }  // namespace chromeos