Remove deprecated extension notifications from APIs

chrome/browser/extensions/api/management/management_api.h

Index: chrome/browser/extensions/api/management/management_api.h
diff --git a/chrome/browser/extensions/api/management/management_api.h b/chrome/browser/extensions/api/management/management_api.h
index 264a08fb5ed40891cac500a4bcaeda4729025fce..84c1c946fb033ef07e2e64b6c975c5a49207068b 100644
--- a/chrome/browser/extensions/api/management/management_api.h
+++ b/chrome/browser/extensions/api/management/management_api.h
@@ -6,6 +6,7 @@
 #define CHROME_BROWSER_EXTENSIONS_API_MANAGEMENT_MANAGEMENT_API_H_
 
 #include "base/compiler_specific.h"
+#include "base/scoped_observer.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "chrome/browser/extensions/bookmark_app_helper.h"
 #include "chrome/browser/extensions/chrome_extension_function.h"
@@ -14,15 +15,15 @@
 #include "chrome/common/web_application_info.h"
 #include "components/favicon_base/favicon_types.h"
 #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/browser_context_keyed_api_factory.h"
 #include "extensions/browser/event_router.h"
+#include "extensions/browser/extension_registry_observer.h"
 
 class ExtensionService;
 class ExtensionUninstallDialog;
 
 namespace extensions {
+class ExtensionRegistry;
 
 class ManagementFunction : public ChromeSyncExtensionFunction {
  protected:
@@ -230,20 +231,31 @@ class ManagementGenerateAppForLinkFunction : public AsyncManagementFunction {
   base::CancelableTaskTracker cancelable_task_tracker_;
 };
 
-class ManagementEventRouter : public content::NotificationObserver {
+class ManagementEventRouter : public ExtensionRegistryObserver {
  public:
-  explicit ManagementEventRouter(Profile* profile);
+  explicit ManagementEventRouter(content::BrowserContext* context);
   virtual ~ManagementEventRouter();
 
  private:
-  // content::NotificationObserver implementation.
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) OVERRIDE;
+  virtual void OnExtensionLoaded(content::BrowserContext* browser_context,
+                                 const Extension* extension) OVERRIDE;
+  virtual void OnExtensionUnloaded(
+      content::BrowserContext* browser_context,
+      const Extension* extension,
+      UnloadedExtensionInfo::Reason reason) OVERRIDE;
+  virtual void OnExtensionWillBeInstalled(

[Devlin, 2014/06/17 16:28:31]

For what this is doing, I think maybe OnExtensionInstalled() is a better fit.

[limasdf, 2014/06/17 18:25:48]

Done.
Yep, Perhaps, There must be many other places to use OnExtensionInstalled()
instead of OnExtensionWillBeInstalled().

+      content::BrowserContext* browser_context,
+      const Extension* extension,
+      bool is_update,
+      bool from_ephemeral,
+      const std::string& old_name) OVERRIDE;
+  virtual void OnExtensionUninstalled(content::BrowserContext* browser_context,
+                                      const Extension* extension) OVERRIDE;
 
-  content::NotificationRegistrar registrar_;
+  content::BrowserContext* browser_context_;
 
-  Profile* profile_;
+  ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
+      extension_registry_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(ManagementEventRouter);
 };