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

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

Issue 2552203007: Public Sessions - prompt the user for pageCapture requests (Closed)
Patch Set: Moved CrOS only code to a separate file Created 3 years, 11 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 ec4db9cd464dfba754cae40fd8a4cdd8b1906930..e5790f407b8134db2e2c6fe42b768d446d55c528 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
@@ -559,9 +559,10 @@ const char* const kSafePermissionStrings[] = {
// icon and won't extract any data.
"notifications",
- // Captures page content, so block. Alternatively: Allow, but either (1)
- // prompt user or (2) return blank content.
- // "pageCapture",
+ // User is prompted (allow/deny) when an extension requests pageCapture for
+ // the first time in a session. The request is made via
+ // chrome.pageCapture.saveAsMHTML call.
+ "pageCapture",
// Allows to use machine crypto keys - these would be provisioned by the
// admin anyways.
« no previous file with comments | « no previous file | chrome/browser/chromeos/preferences.cc » ('j') | chrome/browser/extensions/api/page_capture/page_capture_api.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698