Cleanup/ Use ExtensionRegistry from app_sync_ui_state.*

chrome/browser/ui/ash/app_sync_ui_state.h

Index: chrome/browser/ui/ash/app_sync_ui_state.h
diff --git a/chrome/browser/ui/ash/app_sync_ui_state.h b/chrome/browser/ui/ash/app_sync_ui_state.h
index 958db67c6fc6a1e6f1dc93a6a763a3945126c7c7..5d7c9064c5a3779cb08fb53b20540cabd4e17913 100644
--- a/chrome/browser/ui/ash/app_sync_ui_state.h
+++ b/chrome/browser/ui/ash/app_sync_ui_state.h
@@ -13,18 +13,23 @@
 #include "components/keyed_service/core/keyed_service.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
+#include "extensions/browser/extension_registry_observer.h"
 
 class AppSyncUIStateObserver;
 class Profile;
 class ProfileSyncService;
 
+namespace extensions {
+class ExtensionRegistry;
+}
+
 // AppSyncUIState watches app sync and installation and change its state
 // accordingly. Its status is for UI display only. It only watches for new
 // normal user profile (i.e. it does not watch for guest profile or exsiting
 // user profile) and lasts for at the most 1 minute.
 class AppSyncUIState : public KeyedService,
-                       public content::NotificationObserver,
-                       public ProfileSyncServiceObserver {
+                       public ProfileSyncServiceObserver,
+                       public extensions::ExtensionRegistryObserver {
  public:
   enum Status {
     STATUS_NORMAL,
@@ -62,16 +67,14 @@ class AppSyncUIState : public KeyedService,
   // Invoked when |max_syncing_status_timer_| fires.
   void OnMaxSyncingTimer();
 
-  // content::NotificationObserver overrides:
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) OVERRIDE;
+  // extensions::ExtensionRegistryObserver overrides:

[James Cook, 2014/07/22 16:29:24]

nit: Change this to match the order of the interfaces in the class
AppSyncUIState : public... declaration.  So list ProfileSyncServiceObserver
overrides first.

[limasdf, 2014/07/22 16:38:10]

Done.

+  virtual void OnExtensionLoaded(
+      content::BrowserContext* browser_context,
+      const extensions::Extension* extension) OVERRIDE;
 
   // ProfileSyncServiceObserver overrides:
   virtual void OnStateChanged() OVERRIDE;
 
-  content::NotificationRegistrar registrar_;
-
   Profile* profile_;
   ProfileSyncService* sync_service_;
 
@@ -81,6 +84,8 @@ class AppSyncUIState : public KeyedService,
   Status status_;
   ObserverList<AppSyncUIStateObserver> observers_;
 
+  extensions::ExtensionRegistry* extension_registry_;
+
   DISALLOW_COPY_AND_ASSIGN(AppSyncUIState);
 };