chrome/browser/extensions/extension_service.cc - Issue 2499493004: Communicate ExtensionSettings policy to renderers

Unified Diff: chrome/browser/extensions/extension_service.cc

Issue 2499493004: Communicate ExtensionSettings policy to renderers (Closed)

Patch Set: -Removed old includes, added lock check to SetUsesDefaultHostRestrictions Created 3 years, 9 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

Index: chrome/browser/extensions/extension_service.cc

diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc

index 4cecaddab3e02cc9e9697c9d3f7a01d4ab746468..d50de40f0f060b842777cf58809617d62abf86a9 100644

--- a/chrome/browser/extensions/extension_service.cc

+++ b/chrome/browser/extensions/extension_service.cc

@@ -1212,6 +1212,20 @@ void ExtensionService::CheckManagementPolicy() {

extensions::ExtensionManagement* management =

extensions::ExtensionManagementFactory::GetForBrowserContext(profile());

+ for (const auto& extension : registry_->enabled_extensions()) {

+ bool uses_default =

+ management->UsesDefaultRuntimeHostRestrictions(extension.get());

+ if (!uses_default) {

+ extensions::PermissionsUpdater(profile()).SetPolicyHostRestrictions(

+ extension.get(), management->GetRuntimeBlockedHosts(extension.get()),

+ management->GetRuntimeAllowedHosts(extension.get()));

+ }

+ extensions::PermissionsUpdater(profile()).SetUsesDefaultHostRestrictions(

Devlin 2017/04/03 15:52:23 this should be in an else. As I've mentioned here

nrpeter 2017/04/03 22:35:48 Done.

+ extension.get(), uses_default);

+ }

+ extensions::PermissionsUpdater(profile()).SetDefaultPolicyHostRestrictions(

+ management->GetDefaultRuntimeBlockedHosts(),

+ management->GetDefaultRuntimeAllowedHosts());

// Loop through the disabled extension list, find extensions to re-enable

// automatically. These extensions are exclusive from the |to_disable| and