[MD settings] content category policy

chrome/browser/ui/webui/site_settings_helper.cc

Index: chrome/browser/ui/webui/site_settings_helper.cc
diff --git a/chrome/browser/ui/webui/site_settings_helper.cc b/chrome/browser/ui/webui/site_settings_helper.cc
index 5daa96acbd8328df4d223a2a42e22479db1ec73e..8ee72e247133cd3f0918947e3b53973cbf8fcf2e 100644
--- a/chrome/browser/ui/webui/site_settings_helper.cc
+++ b/chrome/browser/ui/webui/site_settings_helper.cc
@@ -244,6 +244,19 @@ void GetExceptionsFromHostContentSettingsMap(const HostContentSettingsMap* map,
   }
 }
 
+void GetContentCategorySetting(
+    const HostContentSettingsMap* map,
+    ContentSettingsType content_type,
+    base::DictionaryValue* object) {
+  std::string provider;
+  std::string setting = content_settings::ContentSettingToString(
+      map->GetDefaultContentSetting(content_type, &provider));
+  DCHECK(!setting.empty());
+
+  object->SetString(site_settings::kSetting, setting);
+  object->SetString(site_settings::kSource, provider);

[Dan Beam, 2016/11/21 21:48:46]

can we just set this only when provider != default?

[dschuyler, 2016/11/22 02:28:50]

Is that to reduce the size of the IPC call? There are other
uses of the provider source in this api - I'd l ike to 
change them all together (or leave them as is). May I make
that change in a separate CL (and change each call to follow
the same pattern)?

[Dan Beam, 2016/11/22 02:57:40]

no
can you give me an example?  this looks like new code to me...
what does "them all" mean?

[dschuyler, 2016/11/24 01:25:48]

Done.

+}
+
 void GetPolicyAllowedUrls(
     ContentSettingsType type,
     std::vector<std::unique_ptr<base::DictionaryValue>>* exceptions,