Move creation of content TabContentsViews to content so that we can hide these headers through DEPS…

content/public/browser/content_browser_client.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_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #pragma once
 
 #include <string>
 #include <utility>
@@ skipping 52 matching lines @@
 class BrowserContext;
 class BrowserMainParts;
 class RenderProcessHost;
 class QuotaPermissionContext;
 class ResourceContext;
 class RenderViewHost;
 class SiteInstance;
 class SpeechInputManagerDelegate;
 class WebContents;
 class WebContentsView;
+class WebContentsViewDelegate;
 class WebUIControllerFactory;
 struct MainFunctionParams;
 struct ShowDesktopNotificationHostMsgParams;
 
 typedef base::Callback< void(const content::MediaStreamDevices&) >
     MediaResponseCallback;
 
 // Embedder API (or SPI) for participating in browser logic, to be implemented
 // by the client of the content browser. See ChromeContentBrowserClient for the
 // principal implementation. The methods are assumed to be called on the UI
 // thread unless otherwise specified. Use this "escape hatch" sparingly, to
 // avoid the embedder interface ballooning and becoming very specific to Chrome.
 // (Often, the call out to the client can happen in a different part of the code
 // that either already has a hook out to the embedder, or calls out to one of
 // the observer interfaces.)
 class ContentBrowserClient {
  public:
   virtual ~ContentBrowserClient() {}
 
   // Allows the embedder to set any number of custom BrowserMainParts
   // implementations for the browser startup code. See comments in
   // browser_main_parts.h.
   virtual BrowserMainParts* CreateBrowserMainParts(
       const content::MainFunctionParams& parameters) = 0;
 
-  virtual WebContentsView* CreateWebContentsView(WebContents* web_contents) = 0;
+  // Allows an embedder to return their own WebContentsView implementation.
+  // Return NULL to let the default one for the platform be created.
+  virtual WebContentsView* OverrideCreateWebContentsView(
+      WebContents* web_contents) = 0;
+
+  // If content creates the WebContentsView implementation, it will ask the
+  // embedder to return an (optional) delegate to customize it.

[Avi (use Gerrit), 2012/03/14 20:48:24]

You may want to clarify that content will take ownership of a delegate returned
by this call.

[jam, 2012/03/14 20:59:26]

Done.

+  virtual WebContentsViewDelegate* GetWebContentsViewDelegate(
+      WebContents* web_contents) = 0;
 
   // Notifies that a new RenderHostView has been created.
   virtual void RenderViewHostCreated(
       content::RenderViewHost* render_view_host) = 0;
 
   // Notifies that a RenderProcessHost has been created. This is called before
   // the content layer adds its own BrowserMessageFilters, so that the
   // embedder's IPC filters have priority.
   virtual void RenderProcessHostCreated(
       content::RenderProcessHost* host) = 0;
@@ skipping 294 matching lines @@
   // This is called on a worker thread.
   virtual
       crypto::CryptoModuleBlockingPasswordDelegate* GetCryptoPasswordDelegate(
           const GURL& url) = 0;
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_