[Extensions] Update extensions UI to observe events and add test

chrome/browser/ui/webui/extensions/extension_settings_handler.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_SETTINGS_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_SETTINGS_HANDLER_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/scoped_observer.h"
-#include "chrome/browser/extensions/error_console/error_console.h"
 #include "chrome/browser/extensions/extension_management.h"
 #include "chrome/common/extensions/webstore_install_result.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_ui_message_handler.h"
 #include "extensions/browser/extension_prefs_observer.h"
-#include "extensions/browser/extension_registry_observer.h"
 #include "extensions/browser/warning_service.h"
 
 class ExtensionService;
 class GURL;
 
 namespace base {
 class FilePath;
 class ListValue;
 }
 
 namespace content {
 class WebUIDataSource;
 }
 
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 namespace extensions {
 class Extension;
 class ExtensionPrefs;
-class ExtensionRegistry;
 
 // Extension Settings UI handler.
 class ExtensionSettingsHandler
     : public content::WebUIMessageHandler,
       public content::NotificationObserver,
       public content::WebContentsObserver,
-      public ErrorConsole::Observer,
       public ExtensionManagement::Observer,
       public ExtensionPrefsObserver,
-      public ExtensionRegistryObserver,
       public WarningService::Observer,
       public base::SupportsWeakPtr<ExtensionSettingsHandler> {
  public:
   ExtensionSettingsHandler();
   ~ExtensionSettingsHandler() override;
 
   static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);
 
   void GetLocalizedValues(content::WebUIDataSource* source);
 
  private:
   // content::WebContentsObserver implementation.
   void RenderViewDeleted(content::RenderViewHost* render_view_host) override;
   void DidStartNavigationToPendingEntry(
       const GURL& url,
       content::NavigationController::ReloadType reload_type) override;
 
   // WebUIMessageHandler implementation.
   void RegisterMessages() override;
 
-  // ErrorConsole::Observer implementation.
-  void OnErrorAdded(const ExtensionError* error) override;
-
   // content::NotificationObserver implementation.
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override;
 
-  // ExtensionRegistryObserver implementation.
-  void OnExtensionLoaded(content::BrowserContext* browser_context,
-                         const Extension* extension) override;
-  void OnExtensionUnloaded(content::BrowserContext* browser_context,
-                           const Extension* extension,
-                           UnloadedExtensionInfo::Reason reason) override;
-  void OnExtensionUninstalled(content::BrowserContext* browser_context,
-                              const Extension* extension,
-                              extensions::UninstallReason reason) override;
-
   // ExtensionPrefsObserver implementation.
   void OnExtensionDisableReasonsChanged(const std::string& extension_id,
                                         int disable_reasons) override;
 
   // ExtensionManagement::Observer implementation.
   void OnExtensionManagementSettingsChanged() override;
 
   // WarningService::Observer implementation.
   void ExtensionWarningsChanged() override;
 
@@ skipping 64 matching lines @@
   // once to the page, otherwise we'd be calling JavaScript functions on objects
   // that don't exist yet when notifications come in. This variable makes sure
   // we do so only once.
   bool registered_for_notifications_;
 
   content::NotificationRegistrar registrar_;
 
   ScopedObserver<WarningService, WarningService::Observer>
       warning_service_observer_;
 
-  // An observer to listen for when Extension errors are reported.
-  ScopedObserver<ErrorConsole, ErrorConsole::Observer> error_console_observer_;
-
   // An observer to listen for notable changes in the ExtensionPrefs, like
   // a change in Disable Reasons.
   ScopedObserver<ExtensionPrefs, ExtensionPrefsObserver>
       extension_prefs_observer_;
 
-  ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
-      extension_registry_observer_;
-
   ScopedObserver<ExtensionManagement, ExtensionManagement::Observer>
       extension_management_observer_;
 
-  // Whether we found any DISABLE_NOT_VERIFIED extensions and want to kick off
-  // a verification check to try and rescue them.
-  bool should_do_verification_check_;
-
   DISALLOW_COPY_AND_ASSIGN(ExtensionSettingsHandler);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_SETTINGS_HANDLER_H_