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

Side by Side 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 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 // Captures page content, so block. Alternatively: Allow, but either (1) 568 // User is prompted (allow/deny) when an extension requests pageCapture for
569 // prompt user or (2) return blank content. 569 // the first time in a session. The request is made via
570 // "pageCapture", 570 // chrome.pageCapture.saveAsMHTML call.
571 "pageCapture",
571 572
572 // Allows to use machine crypto keys - these would be provisioned by the 573 // Allows to use machine crypto keys - these would be provisioned by the
573 // admin anyways. 574 // admin anyways.
574 "platformKeys", 575 "platformKeys",
575 576
576 // No plugins on Chrome OS anyways. 577 // No plugins on Chrome OS anyways.
577 "plugin", 578 "plugin",
578 579
579 // Status quo notes concern about UX spoofing - not an issue IMHO. 580 // Status quo notes concern about UX spoofing - not an issue IMHO.
580 "pointerLock", 581 "pointerLock",
(...skipping 358 matching lines...) Expand 10 before | Expand all | Expand 10 after
939 if (error) { 940 if (error) {
940 *error = l10n_util::GetStringFUTF16( 941 *error = l10n_util::GetStringFUTF16(
941 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT, 942 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT,
942 base::UTF8ToUTF16(extension->name()), 943 base::UTF8ToUTF16(extension->name()),
943 base::UTF8ToUTF16(extension->id())); 944 base::UTF8ToUTF16(extension->id()));
944 } 945 }
945 return false; 946 return false;
946 } 947 }
947 948
948 } // namespace chromeos 949 } // 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