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

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

Issue 2858643002: PS - Filtering activeTab URL (Closed)
Patch Set: ExtensionBuilder in test Created 3 years, 8 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 7c551d8f5164c304a8bab5d8d7f8ca1e6e2f7bee..92948d8182cedf311ee3577028911272c4b4b7f3 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
@@ -890,8 +890,8 @@ DeviceLocalAccountManagementPolicyProvider::
// static
bool DeviceLocalAccountManagementPolicyProvider::IsWhitelisted(
- const extensions::Extension* extension) {
- return ArrayContains(kPublicSessionWhitelist, extension->id());
+ const std::string& extension_id) {
+ return ArrayContains(kPublicSessionWhitelist, extension_id);
}
std::string DeviceLocalAccountManagementPolicyProvider::
@@ -921,7 +921,7 @@ bool DeviceLocalAccountManagementPolicyProvider::UserMayLoad(
// Allow extension if its specific ID is whitelisted for use in public
// sessions.
- if (IsWhitelisted(extension)) {
+ if (IsWhitelisted(extension->id())) {
return true;
}

Powered by Google App Engine
This is Rietveld 408576698