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

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

Issue 2697833004: Revert of Public Sessions - prompt the user for pageCapture requests (Closed)
Patch Set: 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 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 547 matching lines...) Expand 10 before | Expand all | Expand 10 after
558 "nativeMessaging", 558 "nativeMessaging",
559 559
560 // Admin controls network connectivity anyways. 560 // Admin controls network connectivity anyways.
561 "networking.config", 561 "networking.config",
562 562
563 // Status quo considers this risky due to the ability to fake system UI - 563 // Status quo considers this risky due to the ability to fake system UI -
564 // low risk IMHO however since notifications are already badged with app 564 // low risk IMHO however since notifications are already badged with app
565 // icon and won't extract any data. 565 // icon and won't extract any data.
566 "notifications", 566 "notifications",
567 567
568 // User is prompted (allow/deny) when an extension requests pageCapture for 568 // Captures page content, so block. Alternatively: Allow, but either (1)
569 // the first time in a session. The request is made via 569 // prompt user or (2) return blank content.
570 // chrome.pageCapture.saveAsMHTML call. 570 // "pageCapture",
571 "pageCapture",
572 571
573 // Allows to use machine crypto keys - these would be provisioned by the 572 // Allows to use machine crypto keys - these would be provisioned by the
574 // admin anyways. 573 // admin anyways.
575 "platformKeys", 574 "platformKeys",
576 575
577 // No plugins on Chrome OS anyways. 576 // No plugins on Chrome OS anyways.
578 "plugin", 577 "plugin",
579 578
580 // Status quo notes concern about UX spoofing - not an issue IMHO. 579 // Status quo notes concern about UX spoofing - not an issue IMHO.
581 "pointerLock", 580 "pointerLock",
(...skipping 358 matching lines...) Expand 10 before | Expand all | Expand 10 after
940 if (error) { 939 if (error) {
941 *error = l10n_util::GetStringFUTF16( 940 *error = l10n_util::GetStringFUTF16(
942 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT, 941 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT,
943 base::UTF8ToUTF16(extension->name()), 942 base::UTF8ToUTF16(extension->name()),
944 base::UTF8ToUTF16(extension->id())); 943 base::UTF8ToUTF16(extension->id()));
945 } 944 }
946 return false; 945 return false;
947 } 946 }
948 947
949 } // namespace chromeos 948 } // namespace chromeos
OLDNEW
« 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