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

Unified Diff: chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc

Issue 2794803003: PS - Remove Clipboard Read permission from extensions in Public Sessions (except for whitelisted on… (Closed)
Patch Set: 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:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc
diff --git a/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc b/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc
index 0938b0d2df1e98e429ea4e7ada4f1bf285b0af0a..6fbcaa6504a4488774d4928b800201802cbcb6c4 100644
--- a/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc
+++ b/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc
@@ -889,6 +889,12 @@ DeviceLocalAccountManagementPolicyProvider::
~DeviceLocalAccountManagementPolicyProvider() {
}
+// static
+bool DeviceLocalAccountManagementPolicyProvider::IsWhitelisted(
+ const extensions::Extension* extension) {
+ return ArrayContains(kPublicSessionWhitelist, extension->id());
+}
+
std::string DeviceLocalAccountManagementPolicyProvider::
GetDebugPolicyProviderName() const {
#if defined(NDEBUG)
@@ -916,7 +922,7 @@ bool DeviceLocalAccountManagementPolicyProvider::UserMayLoad(
// Allow extension if its specific ID is whitelisted for use in public
// sessions.
- if (ArrayContains(kPublicSessionWhitelist, extension->id())) {
+ if (IsWhitelisted(extension)) {
return true;
}

Powered by Google App Engine
This is Rietveld 408576698