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

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

Issue 2029263002: [MD Settings] Add feature to enable md-settings by default. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 months 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
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 1085b740a2247975eb87dfb9890a99279c1b5000..6f2d5c3422bf2a1df556be8554826ca9b2abf8e2 100644
--- a/chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc
+++ b/chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc
@@ -410,6 +410,11 @@ WebUIFactoryFunction GetWebUIFactoryFunction(WebUI* web_ui,
url.host() == chrome::kChromeUIHistoryHost) {
return &NewWebUI<MdHistoryUI>;
}
+ // Material Design Settings gets its own host, if enabled.
+ if (base::FeatureList::IsEnabled(features::kMaterialDesignSettingsFeature) &&
+ url.host() == chrome::kChromeUISettingsHost) {
+ return &NewWebUI<settings::MdSettingsUI>;
Dan Beam 2016/06/01 20:59:49 i think it'd make sense to do one of two things he
groby-ooo-7-16 2016/06/01 23:29:35 I'm following the established pattern - all materi
Dan Beam 2016/06/01 23:38:32 ah, i see: UberUI creates it's own WebUIs for the
+ }
if (url.host() == chrome::kChromeUIQuotaInternalsHost)
return &NewWebUI<QuotaInternalsUI>;
// Settings are implemented with native UI elements on Android.

Powered by Google App Engine
This is Rietveld 408576698