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

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: Fixed unit test 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 "audioCapture",
414 // "audioCapture",
415 414
416 // Just resource management, probably doesn't even apply to Chrome OS. 415 // Just resource management, probably doesn't even apply to Chrome OS.
417 "background", 416 "background",
418 417
419 // Access to URLs only, no content. 418 // Access to URLs only, no content.
420 "bookmarks", 419 "bookmarks",
421 420
422 // Open a new tab with a given URL. 421 // Open a new tab with a given URL.
423 "browser", 422 "browser",
424 423
(...skipping 219 matching lines...) Expand 10 before | Expand all | Expand 10 after
644 643
645 // Excessive resource usage is not a risk. 644 // Excessive resource usage is not a risk.
646 "unlimitedStorage", 645 "unlimitedStorage",
647 646
648 // Plugging the USB device is sufficient as consent gesture. 647 // Plugging the USB device is sufficient as consent gesture.
649 "usb", 648 "usb",
650 649
651 // Belongs to the USB API. 650 // Belongs to the USB API.
652 "usbDevices", 651 "usbDevices",
653 652
654 // Need to surface notification to the user. Check what existing UI we have 653 // User is prompted (allow/deny) when an extension requests videoCapture.
655 // and whether that's sufficient for PS. 654 "videoCapture",
656 // "videoCapture",
657 655
658 // Admin controls network config anyways. 656 // Admin controls network config anyways.
659 "vpnProvider", 657 "vpnProvider",
660 658
661 // Just UX. 659 // Just UX.
662 "wallpaper", 660 "wallpaper",
663 661
664 // Privacy sensitive URL access. 662 // Privacy sensitive URL access.
665 // "webNavigation", 663 // "webNavigation",
666 664
(...skipping 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
933 if (error) { 931 if (error) {
934 *error = l10n_util::GetStringFUTF16( 932 *error = l10n_util::GetStringFUTF16(
935 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT, 933 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT,
936 base::UTF8ToUTF16(extension->name()), 934 base::UTF8ToUTF16(extension->name()),
937 base::UTF8ToUTF16(extension->id())); 935 base::UTF8ToUTF16(extension->id()));
938 } 936 }
939 return false; 937 return false;
940 } 938 }
941 939
942 } // namespace chromeos 940 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698