Communicate ExtensionSettings policy to renderers

chrome/browser/extensions/extension_management_unittest.cc

Index: chrome/browser/extensions/extension_management_unittest.cc
diff --git a/chrome/browser/extensions/extension_management_unittest.cc b/chrome/browser/extensions/extension_management_unittest.cc
index 176e7ca8eb47203ce4b8d25b2dc94a09f856eebf..2ad09f11926e88e70921b14153070e8532981b36 100644
--- a/chrome/browser/extensions/extension_management_unittest.cc
+++ b/chrome/browser/extensions/extension_management_unittest.cc
@@ -125,7 +125,7 @@ class ExtensionManagementServiceTest : public testing::Test {
     return extension_management_->global_settings_.get();
   }
 
-  const URLPatternSet& GetRuntimeBlockedHostsById(const std::string& id) {

[Devlin, 2017/01/26 22:47:39]

Why this change?

[nrpeter, 2017/02/03 19:32:24]

It was a syntax issue I didn't detect earlier by compiling unit_tests.

This function isn't actually necessary since RuntimeBlockedHosts is only ever
queried via ID. I went ahead and removed this function and simplified
GetRuntimeBlockedHosts.

+  URLPatternSet GetRuntimeBlockedHostsById(const std::string& id) {
     return GetRuntimeBlockedHosts(id, kNonExistingUpdateUrl);
   }
 
@@ -441,7 +441,7 @@ TEST_F(ExtensionManagementServiceTest, PreferenceParsing) {
             ExtensionManagement::INSTALLATION_BLOCKED);
   EXPECT_EQ(GetInstallationModeByUpdateUrl(kExampleUpdateUrl),
             ExtensionManagement::INSTALLATION_ALLOWED);
-  EXPECT_TRUE(GetRuntimeBlockedHostsById(kTargetExtension).empty());
+  EXPECT_TRUE(GetRuntimeBlockedHostsById(kTargetExtension).is_empty());
   EXPECT_TRUE(GetRuntimeBlockedHostsById(kTargetExtension4)
                   .MatchesURL(GURL("http://test.foo.com/test")));
   EXPECT_TRUE(GetRuntimeBlockedHostsById(kTargetExtension4)