Remove PrefObserver usages, batch 12.

chrome/browser/ui/browser_command_controller.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/ui/browser_command_controller.h"
 
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/defaults.h"
 #include "chrome/browser/extensions/extension_service.h"
@@ skipping 141 matching lines @@
 BrowserCommandController::BrowserCommandController(Browser* browser)
     : browser_(browser),
       ALLOW_THIS_IN_INITIALIZER_LIST(command_updater_(this)),
       block_command_execution_(false),
       last_blocked_command_id_(-1),
       last_blocked_command_disposition_(CURRENT_TAB) {
   browser_->tab_strip_model()->AddObserver(this);
   PrefService* local_state = g_browser_process->local_state();
   if (local_state) {
     local_pref_registrar_.Init(local_state);
-    local_pref_registrar_.Add(prefs::kAllowFileSelectionDialogs, this);
-    local_pref_registrar_.Add(prefs::kInManagedMode, this);
+    local_pref_registrar_.Add(
+        prefs::kAllowFileSelectionDialogs,
+        base::Bind(
+            &BrowserCommandController::UpdateCommandsForFileSelectionDialogs,
+            base::Unretained(this)));
+    local_pref_registrar_.Add(
+        prefs::kInManagedMode,
+        base::Bind(
+            &BrowserCommandController::UpdateCommandsForMultipleProfiles,
+            base::Unretained(this)));
   }
 
   profile_pref_registrar_.Init(profile()->GetPrefs());
-  profile_pref_registrar_.Add(prefs::kDevToolsDisabled, this);
-  profile_pref_registrar_.Add(prefs::kEditBookmarksEnabled, this);
-  profile_pref_registrar_.Add(prefs::kShowBookmarkBar, this);
-  profile_pref_registrar_.Add(prefs::kIncognitoModeAvailability, this);
-  profile_pref_registrar_.Add(prefs::kPrintingEnabled, this);
+  profile_pref_registrar_.Add(
+      prefs::kDevToolsDisabled,
+      base::Bind(&BrowserCommandController::UpdateCommandsForDevTools,
+                 base::Unretained(this)));
+  profile_pref_registrar_.Add(
+      prefs::kEditBookmarksEnabled,
+      base::Bind(&BrowserCommandController::UpdateCommandsForBookmarkEditing,
+                 base::Unretained(this)));
+  profile_pref_registrar_.Add(
+      prefs::kShowBookmarkBar,
+      base::Bind(&BrowserCommandController::UpdateCommandsForBookmarkBar,
+                 base::Unretained(this)));
+  profile_pref_registrar_.Add(
+      prefs::kIncognitoModeAvailability,
+      base::Bind(
+          &BrowserCommandController::UpdateCommandsForIncognitoAvailability,
+          base::Unretained(this)));
+  profile_pref_registrar_.Add(
+      prefs::kPrintingEnabled,
+      base::Bind(&BrowserCommandController::UpdatePrintingState,
+                 base::Unretained(this)));
 
   InitCommandState();
 
   TabRestoreService* tab_restore_service =
       TabRestoreServiceFactory::GetForProfile(profile());
   if (tab_restore_service) {
     tab_restore_service->AddObserver(this);
     TabRestoreServiceChanged(tab_restore_service);
   }
 
@@ skipping 495 matching lines @@
     case content::NOTIFICATION_INTERSTITIAL_DETACHED:
       UpdateCommandsForTabState();
       break;
 
     default:
       NOTREACHED() << "Got a notification we didn't register for.";
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
-// PrefObserver implementation:
-
-void BrowserCommandController::OnPreferenceChanged(
-    PrefServiceBase* service,
-    const std::string& pref_name) {
-  if (pref_name == prefs::kPrintingEnabled) {
-    UpdatePrintingState();
-  } else if (pref_name == prefs::kIncognitoModeAvailability) {
-    UpdateCommandsForIncognitoAvailability();
-  } else if (pref_name == prefs::kDevToolsDisabled) {
-    UpdateCommandsForDevTools();
-  } else if (pref_name == prefs::kEditBookmarksEnabled) {
-    UpdateCommandsForBookmarkEditing();
-  } else if (pref_name == prefs::kShowBookmarkBar) {
-    UpdateCommandsForBookmarkBar();
-  } else if (pref_name == prefs::kAllowFileSelectionDialogs) {
-    UpdateSaveAsState();
-    UpdateOpenFileState();
-  } else if (pref_name == prefs::kInManagedMode) {
-    UpdateCommandsForMultipleProfiles();
-  } else {
-    NOTREACHED();
-  }
-}
-
-////////////////////////////////////////////////////////////////////////////////
 // BrowserCommandController, TabStripModelObserver implementation:
 
 void BrowserCommandController::TabInsertedAt(WebContents* contents,
                                              int index,
                                              bool foreground) {
   AddInterstitialObservers(contents);
 }
 
 void BrowserCommandController::TabDetachedAt(WebContents* contents, int index) {
   RemoveInterstitialObservers(contents);
@@ skipping 322 matching lines @@
 
 void BrowserCommandController::UpdateCommandsForBookmarkBar() {
   const bool show_main_ui =
       IsShowingMainUI(window() && window()->IsFullscreen());
   command_updater_.UpdateCommandEnabled(IDC_SHOW_BOOKMARK_BAR,
       browser_defaults::bookmarks_enabled &&
       !profile()->GetPrefs()->IsManagedPreference(prefs::kShowBookmarkBar) &&
       show_main_ui);
 }
 
+void BrowserCommandController::UpdateCommandsForFileSelectionDialogs() {
+  UpdateSaveAsState();
+  UpdateOpenFileState();
+}
+
 void BrowserCommandController::UpdateCommandsForFullscreenMode(
     FullScreenMode fullscreen_mode) {
   const bool show_main_ui =
       IsShowingMainUI(fullscreen_mode != FULLSCREEN_DISABLED);
   bool main_not_fullscreen = show_main_ui &&
                              (fullscreen_mode == FULLSCREEN_DISABLED);
 
   // Navigation commands
   command_updater_.UpdateCommandEnabled(IDC_OPEN_CURRENT_URL, show_main_ui);
 
@@ skipping 132 matching lines @@
 
 BrowserWindow* BrowserCommandController::window() {
   return browser_->window();
 }
 
 Profile* BrowserCommandController::profile() {
   return browser_->profile();
 }
 
 }  // namespace chrome