Flag for Material Design policy construction page

chrome/browser/ui/webui/policy_ui.cc

Index: chrome/browser/ui/webui/policy_ui.cc
diff --git a/chrome/browser/ui/webui/policy_ui.cc b/chrome/browser/ui/webui/policy_ui.cc
index 7c646e0d19260c0b6334bdd109a985c91ffc81bf..a79255302f449fdcfdb43d225f3424c307ff2147 100644
--- a/chrome/browser/ui/webui/policy_ui.cc
+++ b/chrome/browser/ui/webui/policy_ui.cc
@@ -23,6 +23,7 @@
 #include "chrome/browser/policy/schema_registry_service.h"
 #include "chrome/browser/policy/schema_registry_service_factory.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
 #include "components/policy/core/browser/browser_policy_connector.h"
 #include "components/policy/core/browser/cloud/message_util.h"
@@ -101,11 +102,7 @@ void AddLocalizedPoilcySourceStrings(content::WebUIDataSource* source) {
   }
 }
 
-content::WebUIDataSource* CreatePolicyUIHTMLSource() {
-  content::WebUIDataSource* source =
-      content::WebUIDataSource::Create(chrome::kChromeUIPolicyHost);
-
-  // Localized strings.
+void AddCommonLocalizedStringsToSource(content::WebUIDataSource* source) {
   source->AddLocalizedString("title", IDS_POLICY_TITLE);
   source->AddLocalizedString("filterPlaceholder",
                              IDS_POLICY_FILTER_PLACEHOLDER);
@@ -148,13 +145,26 @@ content::WebUIDataSource* CreatePolicyUIHTMLSource() {
   AddLocalizedPoilcySourceStrings(source);
 
   source->SetJsonPath("strings.js");
+}
+
+content::WebUIDataSource* CreatePolicyMaterialDesignUIHTMLSource() {
+  content::WebUIDataSource* source =
+      content::WebUIDataSource::Create(chrome::kChromeUIMdPolicyHost);
+  AddCommonLocalizedStringsToSource(source);
+  source->SetDefaultResource(IDR_MD_POLICY_HTML);
+  return source;
+}
+
 
+content::WebUIDataSource* CreatePolicyUIHTMLSource() {
+  content::WebUIDataSource* source =
+      content::WebUIDataSource::Create(chrome::kChromeUIPolicyHost);
+  AddCommonLocalizedStringsToSource(source);
   // Add required resources.
   source->AddResourcePath("policy.css", IDR_POLICY_CSS);
   source->AddResourcePath("policy.js", IDR_POLICY_JS);
   source->AddResourcePath("uber_utils.js", IDR_UBER_UTILS_JS);
   source->SetDefaultResource(IDR_POLICY_HTML);
-
   return source;
 }
 
@@ -871,3 +881,13 @@ PolicyUI::PolicyUI(content::WebUI* web_ui) : WebUIController(web_ui) {
 
 PolicyUI::~PolicyUI() {
 }
+
+PolicyMaterialDesignUI::PolicyMaterialDesignUI(content::WebUI* web_ui) :
+    WebUIController(web_ui) {
+  web_ui->AddMessageHandler(new PolicyUIHandler);
+  content::WebUIDataSource::Add(Profile::FromWebUI(web_ui),
+                                CreatePolicyMaterialDesignUIHTMLSource());
+}
+
+PolicyMaterialDesignUI::~PolicyMaterialDesignUI() {

[Thiemo Nagel, 2015/09/25 15:49:30]

What's the purpose of this?

[fhorschig, 2015/09/28 09:38:58]

In case you are pointing to the new WebUIController: I haven't seen a way to
initialize one of those with an extra parameter.

In case you are asking about the destructor: Some childs of WebUIControllwe use
empty destructors, others don't. It compiles without it, so I deleted it for
now.

[Thiemo Nagel, 2015/09/28 12:02:20]

My bad. "Empty" destructors are used to prevent destructor inlining:
https://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Stop-i....

+}