Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3691)

Unified Diff: extensions/browser/management_policy.cc

Issue 536753003: Add recommended extension installation support (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ext-3
Patch Set: rebase, more fix Created 6 years, 2 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:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « extensions/browser/management_policy.h ('k') | extensions/browser/management_policy_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/management_policy.cc
diff --git a/extensions/browser/management_policy.cc b/extensions/browser/management_policy.cc
index 50183d04884cd5596df851ad43c7f58a5aaaf90b..156ba64d456257361a90c0a94107156814b727a4 100644
--- a/extensions/browser/management_policy.cc
+++ b/extensions/browser/management_policy.cc
@@ -47,6 +47,12 @@ bool ManagementPolicy::Provider::MustRemainDisabled(
return false;
}
+bool ManagementPolicy::Provider::MustRemainInstalled(
+ const Extension* extension,
+ base::string16* error) const {
+ return false;
+}
+
void ManagementPolicy::RegisterProvider(Provider* provider) {
providers_.insert(provider);
}
@@ -57,20 +63,20 @@ void ManagementPolicy::UnregisterProvider(Provider* provider) {
bool ManagementPolicy::UserMayLoad(const Extension* extension,
base::string16* error) const {
- return ApplyToProviderList(&Provider::UserMayLoad, "Installation",
- true, extension, error);
+ return ApplyToProviderList(
+ &Provider::UserMayLoad, "Installation", true, extension, error);
}
bool ManagementPolicy::UserMayModifySettings(const Extension* extension,
base::string16* error) const {
- return ApplyToProviderList(&Provider::UserMayModifySettings, "Modification",
- true, extension, error);
+ return ApplyToProviderList(
+ &Provider::UserMayModifySettings, "Modification", true, extension, error);
}
bool ManagementPolicy::MustRemainEnabled(const Extension* extension,
base::string16* error) const {
- return ApplyToProviderList(&Provider::MustRemainEnabled, "Disabling",
- false, extension, error);
+ return ApplyToProviderList(
+ &Provider::MustRemainEnabled, "Disabling", false, extension, error);
}
bool ManagementPolicy::MustRemainDisabled(const Extension* extension,
@@ -84,6 +90,12 @@ bool ManagementPolicy::MustRemainDisabled(const Extension* extension,
return false;
}
+bool ManagementPolicy::MustRemainInstalled(const Extension* extension,
+ base::string16* error) const {
+ return ApplyToProviderList(
+ &Provider::MustRemainInstalled, "Removing", false, extension, error);
+}
+
void ManagementPolicy::UnregisterAllProviders() {
providers_.clear();
}
« no previous file with comments | « extensions/browser/management_policy.h ('k') | extensions/browser/management_policy_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698