Introduces DevToolsManagerDelegate.

content/public/browser/content_browser_client.h

Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index 5f0473ba8649e6312e8392d12c6f9d2e2169aee3..798c813e9c2fde6fb724b2319320a93b15c096a0 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -88,6 +88,7 @@ class BrowserPluginGuestDelegate;
 class BrowserPpapiHost;
 class BrowserURLHandler;
 class DesktopNotificationDelegate;
+class DevToolsManagerDelegate;
 class ExternalVideoSurfaceContainer;
 class LocationProvider;
 class MediaObserver;
@@ -634,6 +635,11 @@ class CONTENT_EXPORT ContentBrowserClient {
   virtual ExternalVideoSurfaceContainer*
   OverrideCreateExternalVideoSurfaceContainer(WebContents* web_contents);
 #endif
+
+  // Creates a new DevToolsManagerDelegate. The caller owns the returned value.
+  // It's valid to return NULL.
+  virtual DevToolsManagerDelegate* CreateDevToolsManagerDelegate(

[pfeldman, 2014/05/12 06:38:24]

I'm looking at GetWebContentsViewDelegate and
GetSpeechRecognitionManagerDelegate and I think it should be called
GetDevToolsManagerDelegate. Returning ownership seems to be common to the
delegate getters.

Since DevToolsManager is singleton, so should be its delegate. And you should
retain this delegate in the devtools manager + expose Inspect in devtools
manager that wires into the delegate.

[horo, 2014/05/12 07:51:38]

Done.

+      BrowserContext* browser_context);
 };
 
 }  // namespace content