Add source column to chrome://policy showing the origins of policies.

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 03d43485804641fe4dced0da2ec12c88bcb51905..dad98ac7966c199ad2d06ad5390ec9431f636c99 100644
--- a/chrome/browser/ui/webui/policy_ui.cc
+++ b/chrome/browser/ui/webui/policy_ui.cc
@@ -77,6 +77,27 @@ namespace em = enterprise_management;
 
 namespace {
 
+struct PolicySourceMap {
+  const char* key;
+  int string_id;
+};
+
+// Strings that map from PolicySource enum to i18n string keys and their IDs.
+const PolicySourceMap kPolicySources[policy::POLICY_SOURCE_COUNT] = {
+  {"", -1},  // policy::POLICY_SOURCE_UNKNOWN

[bartfab (slow), 2015/09/14 14:42:26]

Will the UI handle ID -1 gracefully? Should there not be a separate "unknown"
string?

[fhorschig, 2015/09/16 13:52:04]

The value won't be used as it is mapped to an actual key. the unknown string
would most likely map to an empty string (as discussed offline). Unknown will be
a thing of the past once a source is required for setting a policy.

+  {"sourceEnterprise", IDS_POLICY_SOURCE_ENTERPRISE},
+  {"sourceProgrammatic", IDS_POLICY_SOURCE_PROGRAMMATIC},
+  {"sourcePlatform", IDS_POLICY_SOURCE_PLATFORM},
+  {"sourceCloud", IDS_POLICY_SOURCE_CLOUD}
+};
+
+void AddLocalizedPoilcySourceStrings(content::WebUIDataSource* source) {
+  for (int i = 1; i < policy::POLICY_SOURCE_COUNT; ++i) {
+    source->AddLocalizedString(kPolicySources[i].key,
+                               kPolicySources[i].string_id);
+  }
+}
+
 content::WebUIDataSource* CreatePolicyUIHTMLSource() {
   content::WebUIDataSource* source =
       content::WebUIDataSource::Create(chrome::kChromeUIPolicyHost);
@@ -108,6 +129,7 @@ content::WebUIDataSource* CreatePolicyUIHTMLSource() {
   source->AddLocalizedString("headerName", IDS_POLICY_HEADER_NAME);
   source->AddLocalizedString("headerValue", IDS_POLICY_HEADER_VALUE);
   source->AddLocalizedString("headerStatus", IDS_POLICY_HEADER_STATUS);
+  source->AddLocalizedString("headerSource", IDS_POLICY_HEADER_SOURCE);
   source->AddLocalizedString("showExpandedValue",
                              IDS_POLICY_SHOW_EXPANDED_VALUE);
   source->AddLocalizedString("hideExpandedValue",
@@ -120,6 +142,7 @@ content::WebUIDataSource* CreatePolicyUIHTMLSource() {
   source->AddLocalizedString("unset", IDS_POLICY_UNSET);
   source->AddLocalizedString("unknown", IDS_POLICY_UNKNOWN);
   source->AddLocalizedString("notSpecified", IDS_POLICY_NOT_SPECIFIED);
+  AddLocalizedPoilcySourceStrings(source);
 
   source->SetJsonPath("strings.js");
 
@@ -752,6 +775,7 @@ void PolicyUIHandler::GetPolicyValues(const policy::PolicyMap& map,
       value->SetString("level", "recommended");
     else
       value->SetString("level", "mandatory");
+    value->SetString("source", kPolicySources[entry->second.source].key);
     base::string16 error = errors->GetErrors(entry->first);
     if (!error.empty())
       value->SetString("error", error);