Remove redundant DevToolsManager methods and clean up its clients.

content/public/browser/devtools_manager.h

Index: content/public/browser/devtools_manager.h
diff --git a/content/public/browser/devtools_manager.h b/content/public/browser/devtools_manager.h
index ae5df5a91dcffe0ee5c3902b73d36fd14502d2aa..84b9c9d8ca8a7d6f9e94eeacd1d2e3b7bf107ce6 100644
--- a/content/public/browser/devtools_manager.h
+++ b/content/public/browser/devtools_manager.h
@@ -34,26 +34,27 @@ class CONTENT_EXPORT DevToolsManager {
   virtual bool DispatchOnInspectorBackend(DevToolsClientHost* from,
                                           const std::string& message) = 0;
 
-  // Closes all open developer tools windows.
+  // Routes devtools message from |from| agent host to corresponding
+  // DevToolsClientHost.
+  virtual void DispatchOnInspectorFrontend(DevToolsAgentHost* from,

[pfeldman, 2013/03/29 11:35:49]

I don't think that anyone other than devtools agent should be able to dispatch
messages on agent's behalf.

[Vladislav Kaznacheev, 2013/03/29 15:15:35]

Moved to impl part.
On 2013/03/29 11:35:49, pfeldman wrote:

+                                           const std::string& message) = 0;
+
+  // Disconnects all client hostst.
   virtual void CloseAllClientHosts() = 0;
 
-  // Returns client attached to the |agent_host| if there is one.
-  virtual DevToolsClientHost* GetDevToolsClientHostFor(
-      DevToolsAgentHost* agent_host) = 0;
+  // Returns true if there is a client attached to the |agent_host|.
+  virtual bool IsAttached(DevToolsAgentHost* agent_host) = 0;
 
   // Returns agent that has |client_host| attachd to it if there is one.
   virtual DevToolsAgentHost* GetDevToolsAgentHostFor(
       DevToolsClientHost* client_host) = 0;
 
-  // Registers new DevToolsClientHost for inspected |agent_host|. There must be
-  // no other DevToolsClientHosts registered for the |agent_host| at the moment.
+  // Registers new DevToolsClientHost for inspected |agent_host|. If there is
+  // another DevToolsClientHost registered for the |agent_host| at the moment
+  // it is disconnected.
   virtual void RegisterDevToolsClientHostFor(
       DevToolsAgentHost* agent_host,
       DevToolsClientHost* client_host) = 0;
-  // Unregisters given |agent_host|. DevToolsManager will notify corresponding
-  // client if one is attached.
-  virtual void UnregisterDevToolsClientHostFor(
-      DevToolsAgentHost* agent_host) = 0;
 
   // This method will remove all references from the manager to the
   // DevToolsClientHost and unregister all listeners related to the