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

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: Unused variable 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 541 matching lines...) Expand 10 before | Expand all | Expand 10 after
552 "nativeMessaging", 552 "nativeMessaging",
553 553
554 // Admin controls network connectivity anyways. 554 // Admin controls network connectivity anyways.
555 "networking.config", 555 "networking.config",
556 556
557 // Status quo considers this risky due to the ability to fake system UI - 557 // Status quo considers this risky due to the ability to fake system UI -
558 // low risk IMHO however since notifications are already badged with app 558 // low risk IMHO however since notifications are already badged with app
559 // icon and won't extract any data. 559 // icon and won't extract any data.
560 "notifications", 560 "notifications",
561 561
562 // Captures page content, so block. Alternatively: Allow, but either (1) 562 // User is prompted (allow/deny) when an extension requests pageCapture for
563 // prompt user or (2) return blank content. 563 // the first time in a session. The request is made via
564 // "pageCapture", 564 // chrome.pageCapture.saveAsMHTML call.
565 "pageCapture",
565 566
566 // Allows to use machine crypto keys - these would be provisioned by the 567 // Allows to use machine crypto keys - these would be provisioned by the
567 // admin anyways. 568 // admin anyways.
568 "platformKeys", 569 "platformKeys",
569 570
570 // No plugins on Chrome OS anyways. 571 // No plugins on Chrome OS anyways.
571 "plugin", 572 "plugin",
572 573
573 // Status quo notes concern about UX spoofing - not an issue IMHO. 574 // Status quo notes concern about UX spoofing - not an issue IMHO.
574 "pointerLock", 575 "pointerLock",
(...skipping 358 matching lines...) Expand 10 before | Expand all | Expand 10 after
933 if (error) { 934 if (error) {
934 *error = l10n_util::GetStringFUTF16( 935 *error = l10n_util::GetStringFUTF16(
935 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT, 936 IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT,
936 base::UTF8ToUTF16(extension->name()), 937 base::UTF8ToUTF16(extension->name()),
937 base::UTF8ToUTF16(extension->id())); 938 base::UTF8ToUTF16(extension->id()));
938 } 939 }
939 return false; 940 return false;
940 } 941 }
941 942
942 } // namespace chromeos 943 } // namespace chromeos
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/chromeos/preferences.cc » ('j') | chrome/browser/extensions/api/page_capture/page_capture_api.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698