Remove redundant DevToolsManager methods and clean up its clients.

content/browser/devtools/devtools_manager_impl.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 CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_MANAGER_IMPL_H_
 #define CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_MANAGER_IMPL_H_
 
 #include <map>
 #include <string>
 
@@ skipping 24 matching lines @@
     : public DevToolsAgentHostImpl::CloseListener,
       public DevToolsManager {
  public:
   // Returns single instance of this class. The instance is destroyed on the
   // browser main loop exit so this method MUST NOT be called after that point.
   static DevToolsManagerImpl* GetInstance();
 
   DevToolsManagerImpl();
   virtual ~DevToolsManagerImpl();
 
-  void DispatchOnInspectorFrontend(DevToolsAgentHost* agent_host,
-                                   const std::string& message);
-
   // DevToolsManager implementation
   virtual bool DispatchOnInspectorBackend(DevToolsClientHost* from,
                                           const std::string& message) OVERRIDE;
+  virtual void DispatchOnInspectorFrontend(DevToolsAgentHost* agent_host,
+                                           const std::string& message) OVERRIDE;
+
   virtual void CloseAllClientHosts() OVERRIDE;
-  virtual DevToolsClientHost* GetDevToolsClientHostFor(
-      DevToolsAgentHost* agent_host) OVERRIDE;
+  virtual bool IsAttached(DevToolsAgentHost* agent_host) OVERRIDE;
   virtual DevToolsAgentHost* GetDevToolsAgentHostFor(
       DevToolsClientHost* client_host) OVERRIDE;
   virtual void RegisterDevToolsClientHostFor(
       DevToolsAgentHost* agent_host,
       DevToolsClientHost* client_host) OVERRIDE;
-  virtual void UnregisterDevToolsClientHostFor(
-      DevToolsAgentHost* agent_host) OVERRIDE;
   virtual void ClientHostClosing(DevToolsClientHost* host) OVERRIDE;
 
  private:
   friend struct DefaultSingletonTraits<DevToolsManagerImpl>;
 
   // DevToolsAgentHost::CloseListener implementation.
   virtual void AgentHostClosing(DevToolsAgentHostImpl* host) OVERRIDE;
 
   void BindClientHost(DevToolsAgentHostImpl* agent_host,
                       DevToolsClientHost* client_host);
   void UnbindClientHost(DevToolsAgentHostImpl* agent_host,
                         DevToolsClientHost* client_host);
 
+
+  DevToolsClientHost* GetDevToolsClientHostFor(
+      DevToolsAgentHostImpl* agent_host);
+
+  void UnregisterDevToolsClientHostFor(DevToolsAgentHostImpl* agent_host);
+
   // These two maps are for tracking dependencies between inspected contents and
   // their DevToolsClientHosts. They are useful for routing devtools messages
   // and allow us to have at most one devtools client host per contents.
   //
   // DevToolsManagerImpl starts listening to DevToolsClientHosts when they are
   // put into these maps and removes them when they are closing.
   typedef std::map<DevToolsAgentHostImpl*, DevToolsClientHost*>
       AgentToClientHostMap;
   AgentToClientHostMap agent_to_client_host_;
 
   typedef std::map<DevToolsClientHost*, scoped_refptr<DevToolsAgentHostImpl> >
       ClientToAgentHostMap;
   ClientToAgentHostMap client_to_agent_host_;
 
   DISALLOW_COPY_AND_ASSIGN(DevToolsManagerImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_MANAGER_IMPL_H_