Index: chrome/browser/ui/webui/chrome_web_ui_factory.cc |
=================================================================== |
--- chrome/browser/ui/webui/chrome_web_ui_factory.cc (revision 116777) |
+++ chrome/browser/ui/webui/chrome_web_ui_factory.cc (working copy) |
@@ -49,8 +49,8 @@ |
#include "chrome/common/chrome_switches.h" |
#include "chrome/common/extensions/extension_constants.h" |
#include "chrome/common/url_constants.h" |
-#include "content/browser/tab_contents/tab_contents.h" |
#include "content/browser/webui/web_ui.h" |
+#include "content/public/browser/web_contents.h" |
#include "googleurl/src/gurl.h" |
#if defined(OS_CHROMEOS) |
@@ -84,28 +84,30 @@ |
#include "chrome/browser/ui/webui/aura/app_list_ui.h" |
#endif |
+using content::WebContents; |
+ |
namespace { |
// A function for creating a new WebUI. The caller owns the return value, which |
// may be NULL (for example, if the URL refers to an non-existent extension). |
-typedef WebUI* (*WebUIFactoryFunction)(TabContents* tab_contents, |
+typedef WebUI* (*WebUIFactoryFunction)(WebContents* web_contents, |
const GURL& url); |
// Template for defining WebUIFactoryFunction. |
template<class T> |
-WebUI* NewWebUI(TabContents* contents, const GURL& url) { |
+WebUI* NewWebUI(WebContents* contents, const GURL& url) { |
return new T(contents); |
} |
// Special case for extensions. |
template<> |
-WebUI* NewWebUI<ExtensionWebUI>(TabContents* contents, const GURL& url) { |
+WebUI* NewWebUI<ExtensionWebUI>(WebContents* contents, const GURL& url) { |
return new ExtensionWebUI(contents, url); |
} |
// Special case for older about: handlers. |
template<> |
-WebUI* NewWebUI<AboutUI>(TabContents* contents, const GURL& url) { |
+WebUI* NewWebUI<AboutUI>(WebContents* contents, const GURL& url) { |
return new AboutUI(contents, url.host()); |
} |
@@ -114,25 +116,25 @@ |
// to another container type, like an extension background page. If there is |
// no tab_contents (it's not accessible when calling GetWebUIType and related |
// functions) then we conservatively assume that we need a WebUI. |
-bool NeedsExtensionWebUI(TabContents* tab_contents, |
+bool NeedsExtensionWebUI(WebContents* web_contents, |
Profile* profile, |
const GURL& url) { |
ExtensionService* service = profile ? profile->GetExtensionService() : NULL; |
return service && service->ExtensionBindingsAllowed(url) && |
- (!tab_contents || |
- TabContentsWrapper::GetCurrentWrapperForContents(tab_contents)); |
+ (!web_contents || |
+ TabContentsWrapper::GetCurrentWrapperForContents(web_contents)); |
} |
// Returns a function that can be used to create the right type of WebUI for a |
// tab, based on its URL. Returns NULL if the URL doesn't have WebUI associated |
// with it. |
-WebUIFactoryFunction GetWebUIFactoryFunction(TabContents* tab_contents, |
+WebUIFactoryFunction GetWebUIFactoryFunction(WebContents* web_contents, |
Profile* profile, |
const GURL& url) { |
if (url.host() == chrome::kChromeUIDialogHost) |
return &NewWebUI<ConstrainedHtmlUI>; |
- if (NeedsExtensionWebUI(tab_contents, profile, url)) |
+ if (NeedsExtensionWebUI(web_contents, profile, url)) |
return &NewWebUI<ExtensionWebUI>; |
// All platform builds of Chrome will need to have a cloud printing |
@@ -387,15 +389,15 @@ |
} |
WebUI* ChromeWebUIFactory::CreateWebUIForURL( |
- TabContents* tab_contents, |
+ content::WebContents* web_contents, |
const GURL& url) const { |
Profile* profile = |
- Profile::FromBrowserContext(tab_contents->GetBrowserContext()); |
- WebUIFactoryFunction function = GetWebUIFactoryFunction(tab_contents, |
+ Profile::FromBrowserContext(web_contents->GetBrowserContext()); |
+ WebUIFactoryFunction function = GetWebUIFactoryFunction(web_contents, |
profile, url); |
if (!function) |
return NULL; |
- return (*function)(tab_contents, url); |
+ return (*function)(web_contents, url); |
} |
void ChromeWebUIFactory::GetFaviconForURL( |