Communicate ExtensionSettings policy to renderers

chrome/browser/extensions/extension_management.cc

Index: chrome/browser/extensions/extension_management.cc
diff --git a/chrome/browser/extensions/extension_management.cc b/chrome/browser/extensions/extension_management.cc
index 727d20c99088bfa2df855f0331e652dee59fe548..c39cbee27ec958d44faa5442d4d67691b8ee003f 100644
--- a/chrome/browser/extensions/extension_management.cc
+++ b/chrome/browser/extensions/extension_management.cc
@@ -211,6 +211,16 @@ APIPermissionSet ExtensionManagement::GetBlockedAPIPermissions(
   return default_settings_->blocked_permissions;
 }
 
+const URLPatternSet& ExtensionManagement::GetDefaultRuntimeBlockedHosts()
+    const {
+  return default_settings_->runtime_blocked_hosts;
+}
+
+const URLPatternSet& ExtensionManagement::GetDefaultRuntimeAllowedHosts()
+    const {
+  return default_settings_->runtime_allowed_hosts;
+}
+
 const URLPatternSet& ExtensionManagement::GetRuntimeBlockedHosts(
     const Extension* extension) const {
   auto iter_id = settings_by_id_.find(extension->id());
@@ -227,8 +237,13 @@ const URLPatternSet& ExtensionManagement::GetRuntimeAllowedHosts(
   return default_settings_->runtime_allowed_hosts;
 }
 
-bool ExtensionManagement::IsBlockedHost(const Extension* extension,
-                                        const GURL& url) const {
+bool ExtensionManagement::UsesDefaultRuntimeHostRestrictions(
+    const Extension* extension) const {

[dcheng, 2017/02/06 07:05:29]

Pass by const reference if it's never null? *shrug*

[nrpeter, 2017/02/06 22:53:15]

Everything else in extension_management that passes an extension uses const *.
Would we rather have consistency or switch this to const &? I'm fine either way,
just let me know.

[dcheng, 2017/02/07 07:47:12]

I guess I'll defer to devlin on this one, but that's my preference (even if it
means it'll take some time to converge on a consistent style here).

+  return (settings_by_id_.find(extension->id()) == settings_by_id_.end());
+}
+
+bool ExtensionManagement::IsRuntimeBlockedHost(const Extension* extension,
+                                               const GURL& url) const {
   auto iter_id = settings_by_id_.find(extension->id());
   if (iter_id != settings_by_id_.end())
     return iter_id->second->runtime_blocked_hosts.MatchesURL(url);