Removing PrefObserver usage, batch 4.

chrome/browser/extensions/extension_service.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/extensions/extension_service.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 369 matching lines @@
                  content::NotificationService::AllBrowserContextsAndSources());
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_PROCESS_TERMINATED,
                  content::NotificationService::AllBrowserContextsAndSources());
   registrar_.Add(this, content::NOTIFICATION_RENDERER_PROCESS_CREATED,
                  content::NotificationService::AllBrowserContextsAndSources());
   registrar_.Add(this, content::NOTIFICATION_RENDERER_PROCESS_TERMINATED,
                  content::NotificationService::AllBrowserContextsAndSources());
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_HOST_DESTROYED,
                  content::NotificationService::AllBrowserContextsAndSources());
   pref_change_registrar_.Init(profile->GetPrefs());
-  pref_change_registrar_.Add(prefs::kExtensionInstallAllowList, this);
-  pref_change_registrar_.Add(prefs::kExtensionInstallDenyList, this);
+  base::Closure callback =
+      base::Bind(&ExtensionService::OnExtensionInstallPrefChanged,
+                 base::Unretained(this));
+  pref_change_registrar_.Add(prefs::kExtensionInstallAllowList, callback);
+  pref_change_registrar_.Add(prefs::kExtensionInstallDenyList, callback);
 
   // Set up the ExtensionUpdater
   if (autoupdate_enabled) {
     int update_frequency = kDefaultUpdateFrequencySeconds;
     if (command_line->HasSwitch(switches::kExtensionsUpdateFrequency)) {
       base::StringToInt(command_line->GetSwitchValueASCII(
           switches::kExtensionsUpdateFrequency),
           &update_frequency);
     }
     updater_.reset(new extensions::ExtensionUpdater(this,
@@ skipping 2301 matching lines @@
         }
       }
       break;
     }
 
     default:
       NOTREACHED() << "Unexpected notification type.";
   }
 }
 
-void ExtensionService::OnPreferenceChanged(PrefServiceBase* service,
-                                           const std::string& pref_name) {
-  DCHECK(pref_name == prefs::kExtensionInstallAllowList ||
-         pref_name == prefs::kExtensionInstallDenyList)
-      << "Unexpected preference name " << pref_name;
+void ExtensionService::OnExtensionInstallPrefChanged() {
   IdentifyAlertableExtensions();
   CheckManagementPolicy();
 }
 
 bool ExtensionService::HasApps() const {
   return !GetAppIds().empty();
 }
 
 ExtensionIdSet ExtensionService::GetAppIds() const {
   ExtensionIdSet result;
@@ skipping 227 matching lines @@
 
   // If there is no ExtensionProcessManager then assume the process is idle.
   // This is commonly the case in unit-tests.
   if (process_manager == NULL)
     return true;
 
   extensions::ExtensionHost* host =
       process_manager->GetBackgroundHostForExtension(extension_id);
   return !host;
 }