Introduces DevToolsManagerDelegate.

content/public/browser/devtools_manager.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_PUBLIC_BROWSER_DEVTOOLS_MANAGER_H_
 #define CONTENT_PUBLIC_BROWSER_DEVTOOLS_MANAGER_H_
 
 #include <string>
 
 #include "base/callback.h"
 #include "content/common/content_export.h"
 
 namespace IPC {
 class Message;
 }
 
 namespace content {
 
+class BrowserContext;
 class DevToolsAgentHost;
 class DevToolsClientHost;
 
 // DevToolsManager connects a devtools client to inspected agent and routes
 // devtools messages between the inspected instance represented by the agent
 // and devtools front-end represented by the client. If inspected agent
 // gets terminated DevToolsManager will notify corresponding client and
 // remove it from the map.
 class CONTENT_EXPORT DevToolsManager {
  public:
@@ skipping 22 matching lines @@
 
   // This method will remove all references from the manager to the
   // DevToolsClientHost and unregister all listeners related to the
   // DevToolsClientHost. Called by closing client.
   virtual void ClientHostClosing(DevToolsClientHost* client_host) = 0;
 
   typedef base::Callback<void(DevToolsAgentHost*, bool attached)> Callback;
 
   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
 
 #endif  // CONTENT_PUBLIC_BROWSER_DEVTOOLS_MANAGER_H_