[DevTools] Migrate from AgentStateCallbacks to DevToolsAgentHostObserver.

extensions/browser/app_window/app_window_registry.h

 // Copyright 2014 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 EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_REGISTRY_H_
 #define EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_REGISTRY_H_
 
 #include <list>
 #include <set>
 #include <string>
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/memory/singleton.h"
 #include "base/observer_list.h"
 #include "components/keyed_service/content/browser_context_keyed_service_factory.h"
 #include "components/keyed_service/core/keyed_service.h"
+#include "content/public/browser/devtools_agent_host_observer.h"
 #include "ui/gfx/native_widget_types.h"
 
 namespace content {
 class BrowserContext;
 class DevToolsAgentHost;
 class WebContents;
 }
 
 namespace extensions {
 
 class AppWindow;
 
 // The AppWindowRegistry tracks the AppWindows for all platform apps for a
 // particular browser context.
-class AppWindowRegistry : public KeyedService {
+class AppWindowRegistry : public KeyedService,
+                          public content::DevToolsAgentHostObserver {
  public:
   class Observer {
    public:
     // Called just after a app window was added.
     virtual void OnAppWindowAdded(AppWindow* app_window);
     // Called when the window icon changes.
     virtual void OnAppWindowIconChanged(AppWindow* app_window);
     // Called just after a app window was removed.
     virtual void OnAppWindowRemoved(AppWindow* app_window);
     // Called just after a app window was hidden. This is different from
@@ skipping 75 matching lines @@
 
     // BrowserContextKeyedServiceFactory
     KeyedService* BuildServiceInstanceFor(
         content::BrowserContext* context) const override;
     bool ServiceIsCreatedWithBrowserContext() const override;
     bool ServiceIsNULLWhileTesting() const override;
     content::BrowserContext* GetBrowserContextToUse(
         content::BrowserContext* context) const override;
   };
 
- protected:
-  void OnDevToolsStateChanged(content::DevToolsAgentHost*, bool attached);
-
  private:
   // Ensures the specified |app_window| is included in |app_windows_|.
   // Otherwise adds |app_window| to the back of |app_windows_|.
   void AddAppWindowToList(AppWindow* app_window);
 
   // Bring |app_window| to the front of |app_windows_|. If it is not in the
   // list, add it first.
   void BringToFront(AppWindow* app_window);
 
   // Create a key that identifies an AppWindow across App reloads. If the window
   // was given an id in CreateParams, the key is the extension id, a colon
   // separator, and the AppWindow's |id|. If there is no |id|, the
   // chrome-extension://extension-id/page.html URL will be used. If the
   // WebContents is not for a AppWindow, return an empty string.
   std::string GetWindowKeyForWebContents(
       content::WebContents* web_contents) const;
+  std::string GetWindowKeyForAgentHost(
+      content::DevToolsAgentHost* agent_host) const;
+
+  // content::DevToolsAgentHostObserver overrides.
+  void DevToolsAgentHostAttached(
+      content::DevToolsAgentHost* agent_host) override;
+  void DevToolsAgentHostDetached(
+      content::DevToolsAgentHost* agent_host) override;
 
   content::BrowserContext* context_;
   AppWindowList app_windows_;
   InspectedWindowSet inspected_windows_;
   base::ObserverList<Observer> observers_;
-  base::Callback<void(content::DevToolsAgentHost*, bool)> devtools_callback_;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_APP_WINDOW_APP_WINDOW_REGISTRY_H_