Remove content::NotificationObserver dependency from most Prefs code.

chrome/browser/signin/signin_manager.cc

Index: chrome/browser/signin/signin_manager.cc
diff --git a/chrome/browser/signin/signin_manager.cc b/chrome/browser/signin/signin_manager.cc
index b6ac90f6eb0922c25601cc760a94fd0fcd6671c0..2f8c6fe50b1cd961bdc566beef556f1a8fa94848 100644
--- a/chrome/browser/signin/signin_manager.cc
+++ b/chrome/browser/signin/signin_manager.cc
@@ -497,17 +497,6 @@ void SigninManager::Observe(int type,
                             const content::NotificationSource& source,
                             const content::NotificationDetails& details) {
   switch (type) {
-    case chrome::NOTIFICATION_PREF_CHANGED:
-      DCHECK(*content::Details<std::string>(details).ptr() ==
-             prefs::kGoogleServicesUsernamePattern);
-      if (!authenticated_username_.empty() &&
-          !IsAllowedUsername(authenticated_username_)) {
-        // Signed in user is invalid according to the current policy so sign
-        // the user out.
-        SignOut();
-      }
-      break;
-
 #if !defined(OS_CHROMEOS)
     case chrome::NOTIFICATION_TOKEN_AVAILABLE: {
       TokenService::TokenAvailableDetails* tok_details =
@@ -537,3 +526,13 @@ void SigninManager::Observe(int type,
   }
 }
 
+void SigninManager::OnPreferenceChanged(PrefServiceBase* service,
+                                        const std::string& pref_name) {
+  DCHECK_EQ(prefs::kGoogleServicesUsernamePattern, pref_name);
+  if (!authenticated_username_.empty() &&
+      !IsAllowedUsername(authenticated_username_)) {
+    // Signed in user is invalid according to the current policy so sign
+    // the user out.
+    SignOut();
+  }
+}