Introduces DevToolsManagerDelegate.

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 32082ea8bc55a24c1ffea5d1e2e8dcc0b5718352..3deadaf8b65a3e653e433fa7ec39e2f89a1b5324 100644
--- a/content/public/browser/devtools_manager.h
+++ b/content/public/browser/devtools_manager.h
@@ -16,6 +16,7 @@ class Message;
 
 namespace content {
 
+class BrowserContext;
 class DevToolsAgentHost;
 class DevToolsClientHost;
 
@@ -58,6 +59,10 @@ class CONTENT_EXPORT DevToolsManager {
 
   virtual void AddAgentStateCallback(const Callback& callback) = 0;
   virtual void RemoveAgentStateCallback(const Callback& callback) = 0;
+
+  // Opens the inspector for |agent_host|.
+  virtual void Inspect(BrowserContext* browser_context,

[jam, 2014/05/12 15:01:23]

why is there a method on both DevToolsManager & DevToolsManagerDelegate?

please include the chrome code which calls this in this cl, otherwise it's hard
to review content api changes to know what is necessary and what is not

[horo, 2014/05/12 16:21:48]

Oh, we don't need this.
Removed Inspect() from DevToolsManager.

+                       DevToolsAgentHost* agent_host) = 0;
 };
 
 }  // namespace content