Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3091)

Unified Diff: chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc

Issue 2468673003: [Extensions] Remove ExtensionWebUI (Closed)
Patch Set: nit Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/extensions/tab_helper.cc ('k') | content/browser/frame_host/navigator_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc
diff --git a/chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc b/chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc
index 00eb74009119f9145367a58777b7e8d6b4e1e25d..14d2961207aafade699112c767f611de3a86d8a0 100644
--- a/chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc
+++ b/chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc
@@ -216,15 +216,6 @@ WebUIController* NewWebUI(WebUI* web_ui, const GURL& url) {
return new T(web_ui);
}
-#if defined(ENABLE_EXTENSIONS)
-// Special cases for extensions.
-template<>
-WebUIController* NewWebUI<ExtensionWebUI>(WebUI* web_ui,
- const GURL& url) {
- return new ExtensionWebUI(web_ui, url);
-}
-#endif // defined(ENABLE_EXTENSIONS)
-
// Special case for older about: handlers.
template<>
WebUIController* NewWebUI<AboutUI>(WebUI* web_ui, const GURL& url) {
@@ -312,11 +303,6 @@ bool IsAboutUI(const GURL& url) {
WebUIFactoryFunction GetWebUIFactoryFunction(WebUI* web_ui,
Profile* profile,
const GURL& url) {
-#if defined(ENABLE_EXTENSIONS)
- if (ExtensionWebUI::NeedsExtensionWebUI(profile, url))
- return &NewWebUI<ExtensionWebUI>;
-#endif
-
// This will get called a lot to check all URLs, so do a quick check of other
// schemes to filter out most URLs.
if (!url.SchemeIs(content::kChromeDevToolsScheme) &&
@@ -671,14 +657,7 @@ bool ChromeWebUIControllerFactory::UseWebUIForURL(
bool ChromeWebUIControllerFactory::UseWebUIBindingsForURL(
content::BrowserContext* browser_context, const GURL& url) const {
- bool needs_extensions_web_ui = false;
-#if defined(ENABLE_EXTENSIONS)
- // Extensions are rendered via WebUI in tabs, but don't actually need WebUI
- // bindings (see the ExtensionWebUI constructor).
- needs_extensions_web_ui =
- ExtensionWebUI::NeedsExtensionWebUI(browser_context, url);
-#endif
- return !needs_extensions_web_ui && UseWebUIForURL(browser_context, url);
+ return UseWebUIForURL(browser_context, url);
}
WebUIController* ChromeWebUIControllerFactory::CreateWebUIControllerForURL(
« no previous file with comments | « chrome/browser/extensions/tab_helper.cc ('k') | content/browser/frame_host/navigator_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698