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

Move creation of content TabContentsViews to content so that we can hide these headers through DEPS and avoid any more includes until TabContentsViewsViews is removed. I also moved all the TCVXDelegate to be in one header, that way we can more conveniently ask the embedder for a delegate without a lot of ifdefs.

BUG=98716
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126730

[jam, 2012-03-14 16:59:33]

I'll remove the tab_contents_view_gtk.h include in a followup change, I need to
make the delegate for gtk handle sad tab/constrained windows like the windows
version.

[jam, 2012-03-14 19:53:21]

->Avi

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

A small note about clarifying ownership semantics, but it lgtm.

http://codereview.chromium.org/9700023/diff/8001/content/public/browser/conte...
File content/public/browser/content_browser_client.h (right):

http://codereview.chromium.org/9700023/diff/8001/content/public/browser/conte...
content/public/browser/content_browser_client.h:105: // embedder to return an
(optional) delegate to customize it.
You may want to clarify that content will take ownership of a delegate returned
by this call.

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

http://codereview.chromium.org/9700023/diff/8001/content/public/browser/conte...
File content/public/browser/content_browser_client.h (right):

http://codereview.chromium.org/9700023/diff/8001/content/public/browser/conte...
content/public/browser/content_browser_client.h:105: // embedder to return an
(optional) delegate to customize it.
On 2012/03/14 20:48:24, Avi wrote:
> You may want to clarify that content will take ownership of a delegate
returned
> by this call.

Done.

[Jói, 2012-03-15 12:41:00]

LGTM