PS - Remove Clipboard Read permission from extensions in Public Sessions (except for whitelisted on…

chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc

+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.h"
+
+#include "chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.h"
+#include "chrome/browser/profiles/profiles_state.h"
+#include "extensions/common/permissions/api_permission.h"
+#include "extensions/common/permissions/api_permission_set.h"
+#include "extensions/common/permissions/manifest_permission_set.h"
+#include "extensions/common/permissions/permission_set.h"
+#include "extensions/common/url_pattern_set.h"
+
+namespace extensions {
+
+PermissionsUpdaterDelegateChromeOS::PermissionsUpdaterDelegateChromeOS() {}
+
+PermissionsUpdaterDelegateChromeOS::~PermissionsUpdaterDelegateChromeOS() {}
+
+void PermissionsUpdaterDelegateChromeOS::InitializePermissions(
+    const Extension* extension,
+    std::unique_ptr<const PermissionSet>* granted_permissions) {
+  if (!profiles::IsPublicSession() ||
+      chromeos::DeviceLocalAccountManagementPolicyProvider::IsWhitelisted(
+          extension)) {

[Devlin, 2017/04/08 00:30:51]

nit: || !granted_permissions->HasAPIPermission(APIPermission::kClipboardRead)

[Ivan Šandrk, 2017/04/10 12:04:35]

Done.

+    return;
+  }
+  // Revoke kClipboardRead permission (used in Public Sessions to secure
+  // clipboard read functionality).
+  APIPermissionSet api_permission_set((*granted_permissions)->apis());
+  api_permission_set.erase(APIPermission::kClipboardRead);
+  granted_permissions->reset(
+      new PermissionSet(api_permission_set, ManifestPermissionSet(),
+                        URLPatternSet(), URLPatternSet()));
+}
+
+}  // namespace extensions