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..3f080235744488fc72d24e7f85223ea6d10b368a 100644
--- a/content/public/browser/devtools_manager.h
+++ b/content/public/browser/devtools_manager.h
@@ -34,26 +34,22 @@ class CONTENT_EXPORT DevToolsManager {
   virtual bool DispatchOnInspectorBackend(DevToolsClientHost* from,
                                           const std::string& message) = 0;
 
-  // Closes all open developer tools windows.
+  // 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;

[pfeldman, 2013/03/29 14:59:23]

Sounds like an instance method on DevToolsAgentHost

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

The real knowledge resides in DevToolsManager(Impl) and if we add this to
DevToolsAgentHost it will be a convenience method.
The whole point of this change was removing a public GetDevToolsClientHostFor.
On 2013/03/29 14:59:23, pfeldman wrote:

[pfeldman, 2013/03/30 08:54:57]

Here is how I see it: you are adding this method into the public API. It always
accepts DTAH that resides right next to DTM. All I am suggesting is to move it
there so that call sites were simpler. We should not consider DTM API separate
from DTAH. They are two pieces of one puzzle, the API the expose is a single
entity and I'm suggesting structuring it better.

[Vladislav Kaznacheev, 2013/04/01 09:26:17]

Agree. Done.

On 2013/03/30 08:54:57, pfeldman wrote:

 
   // 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