Public Sessions - prompt the user for audioCapture/videoCapture requests

chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc

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..6b6f25d3d89977be3ab35689ce3e7a2b5c14eba3 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
@@ -409,9 +409,8 @@ const char* const kSafePermissionStrings[] = {
     // Risk of listening attack.
     // "audio",
 
-    // Need to surface notification to the user. Check what existing UI we have
-    // and whether that's sufficient for PS.
-    // "audioCapture",
+    // User is prompted (allow/deny) when an extension requests audioCapture.

[Andrew T Wilson (Slow), 2016/12/04 19:57:01]

Clarify this - "the user is prompted when the extension calls <some API>" -
explicitly state which API, both here and for videoCapture.

[Ivan Šandrk, 2016/12/05 17:04:46]

Done.

+    "audioCapture",
 
     // Just resource management, probably doesn't even apply to Chrome OS.
     "background",
@@ -651,9 +650,8 @@ const char* const kSafePermissionStrings[] = {
     // Belongs to the USB API.
     "usbDevices",
 
-    // Need to surface notification to the user. Check what existing UI we have
-    // and whether that's sufficient for PS.
-    // "videoCapture",
+    // User is prompted (allow/deny) when an extension requests videoCapture.
+    "videoCapture",
 
     // Admin controls network config anyways.
     "vpnProvider",