Remove content::NotificationObserver dependency from most Prefs code.

chrome/browser/chromeos/proxy_config_service_impl.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/proxy_config_service_impl.h"
 
 #include <ostream>
 
 #include "base/bind.h"
 #include "base/json/json_string_value_serializer.h"
@@ skipping 568 matching lines @@
 void ProxyConfigServiceImpl::RegisterPrefs(PrefService* pref_service) {
   // Use shared proxies default to off.  GetUseSharedProxies will return the
   // correct value based on pre-login and login.
   pref_service->RegisterBooleanPref(prefs::kUseSharedProxies,
                                     true,
                                     PrefService::UNSYNCABLE_PREF);
 }
 
 //------------------ ProxyConfigServiceImpl: private methods -------------------
 
-void ProxyConfigServiceImpl::Observe(
-    int type,
-    const content::NotificationSource& source,
-    const content::NotificationDetails& details) {
-  if (type == chrome::NOTIFICATION_PREF_CHANGED &&
-      *(content::Details<std::string>(details).ptr()) ==
-          prefs::kUseSharedProxies) {
-    if (content::Source<PrefService>(source).ptr() == prefs()) {
-      VLOG(1) << "New use-shared-proxies = " << GetUseSharedProxies();
-      // Determine new proxy config which may have changed because of new
-      // use-shared-proxies. If necessary, activate it.
-      Network* network = NULL;
-      if (!active_network_.empty()) {
-        network = CrosLibrary::Get()->GetNetworkLibrary()->FindNetworkByPath(
-            active_network_);
-        if (!network)
-          LOG(WARNING) << "Can't find requested network " << active_network_;
-      }
-      DetermineEffectiveConfig(network, true);
+void ProxyConfigServiceImpl::OnPreferenceChanged(PrefServiceBase* service,
+                                                 const std::string& pref_name) {
+  DCHECK(service == prefs());
+  VLOG(1) << "New use-shared-proxies = " << GetUseSharedProxies();
+
+  if (pref_name == prefs::kUseSharedProxies) {
+    // Determine new proxy config which may have changed because of new
+    // use-shared-proxies. If necessary, activate it.
+    Network* network = NULL;
+    if (!active_network_.empty()) {
+      network = CrosLibrary::Get()->GetNetworkLibrary()->FindNetworkByPath(
+          active_network_);
+      if (!network)
+        LOG(WARNING) << "Can't find requested network " << active_network_;
     }
+    DetermineEffectiveConfig(network, true);
     return;
   }
-  PrefProxyConfigTrackerImpl::Observe(type, source, details);
+
+  PrefProxyConfigTrackerImpl::OnPreferenceChanged(service, pref_name);
 }
 
 void ProxyConfigServiceImpl::OnUISetProxyConfig() {
   if (current_ui_network_.empty())
     return;
   // Update config to shill.
   std::string value;
   if (current_ui_config_.SerializeForNetwork(&value)) {
     VLOG(1) << "Set proxy (mode=" << current_ui_config_.mode
             << ") for " << current_ui_network_;
@@ skipping 224 matching lines @@
     device_config_.clear();
     return;
   }
   if (!active_network_.empty()) {
     VLOG(1) << "Try migrating device config to " << active_network_;
     SetProxyConfigForNetwork(active_network_, device_config_, true);
   }
 }
 
 }  // namespace chromeos