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

Side by Side Diff: chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc

Issue 2532323003: Public Sessions - prompt the user for audioCapture/videoCapture requests (Closed)
Patch Set: std::move, formatting Created 4 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/extensions/device_local_account_management_pol icy_provider.h" 5 #include "chrome/browser/chromeos/extensions/device_local_account_management_pol icy_provider.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <cstddef> 9 #include <cstddef>
10 #include <string> 10 #include <string>
(...skipping 391 matching lines...) Expand 10 before | Expand all | Expand 10 after
402 "overrideEscFullscreen", 402 "overrideEscFullscreen",
403 403
404 "app.window.shape", 404 "app.window.shape",
405 405
406 // The embedded app is subject to the restrictions as well obviously. 406 // The embedded app is subject to the restrictions as well obviously.
407 "appview", 407 "appview",
408 408
409 // Risk of listening attack. 409 // Risk of listening attack.
410 // "audio", 410 // "audio",
411 411
412 // Need to surface notification to the user. Check what existing UI we have 412 // User is prompted (allow/deny) when an extension requests audioCapture.
413 // and whether that's sufficient for PS. 413 // The request is done via the getUserMedia API.
414 // "audioCapture", 414 "audioCapture",
415 415
416 // Just resource management, probably doesn't even apply to Chrome OS. 416 // Just resource management, probably doesn't even apply to Chrome OS.
417 "background", 417 "background",
418 418
419 // Access to URLs only, no content. 419 // Access to URLs only, no content.
420 "bookmarks", 420 "bookmarks",
421 421
422 // Open a new tab with a given URL. 422 // Open a new tab with a given URL.
423 "browser", 423 "browser",
424 424
(...skipping 219 matching lines...) Expand 10 before | Expand all | Expand 10 after
644 644
645 // Excessive resource usage is not a risk. 645 // Excessive resource usage is not a risk.
646 "unlimitedStorage", 646 "unlimitedStorage",
647 647
648 // Plugging the USB device is sufficient as consent gesture. 648 // Plugging the USB device is sufficient as consent gesture.
649 "usb", 649 "usb",
650 650
651 // Belongs to the USB API. 651 // Belongs to the USB API.
652 "usbDevices", 652 "usbDevices",
653 653
654 // Need to surface notification to the user. Check what existing UI we have 654 // User is prompted (allow/deny) when an extension requests videoCapture.
655 // and whether that's sufficient for PS. 655 // The request is done via the getUserMedia API.
656 // "videoCapture", 656 "videoCapture",
657 657
658 // Admin controls network config anyways. 658 // Admin controls network config anyways.
659 "vpnProvider", 659 "vpnProvider",
660 660
661 // Just UX. 661 // Just UX.
662 "wallpaper", 662 "wallpaper",
663 663
664 // Privacy sensitive URL access. 664 // Privacy sensitive URL access.
665 // "webNavigation", 665 // "webNavigation",
666 666
(...skipping 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
933 if (error) { 933 if (error) {
934 *error = l10n_util::GetStringFUTF16( 934 *error = l10n_util::GetStringFUTF16(
935 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT, 935 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT,
936 base::UTF8ToUTF16(extension->name()), 936 base::UTF8ToUTF16(extension->name()),
937 base::UTF8ToUTF16(extension->id())); 937 base::UTF8ToUTF16(extension->id()));
938 } 938 }
939 return false; 939 return false;
940 } 940 }
941 941
942 } // namespace chromeos 942 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698