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

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: Nitfix Created 3 years, 10 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 d73b3878951ba5ad34aaf941a64003b090a73add..0938b0d2df1e98e429ea4e7ada4f1bf285b0af0a 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
@@ -565,9 +565,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 | « chrome/browser/chromeos/BUILD.gn ('k') | chrome/browser/chromeos/extensions/public_session_permission_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698