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

Unified Diff: chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc

Issue 2840043002: Fix for broken "Load unpacked extension" popup (Closed)
Patch Set: TODO, CreatePermissions with a parameter Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
diff --git a/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc b/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
index 28af4c8469ba2a3acf05fc87be106e56fc9c6ea0..f6ac7f4c20f108f6740f56a591de3e9bb6c8e53d 100644
--- a/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
+++ b/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
@@ -32,11 +32,12 @@ void PermissionsUpdaterDelegateChromeOS::InitializePermissions(
// clipboard read functionality). This forceful removal of permission is safe
// since the clipboard pasting code checks for this permission before doing
// the paste (the end result is just an empty paste).
- APIPermissionSet api_permission_set((*granted_permissions)->apis());
- api_permission_set.erase(APIPermission::kClipboardRead);
- granted_permissions->reset(
- new PermissionSet(api_permission_set, ManifestPermissionSet(),
- URLPatternSet(), URLPatternSet()));
+ APIPermissionSet api_permission_set;
+ api_permission_set.insert(APIPermission::kClipboardRead);
+ *granted_permissions = PermissionSet::CreateDifference(
+ **granted_permissions,
+ PermissionSet(api_permission_set, ManifestPermissionSet(),
+ URLPatternSet(), URLPatternSet()));
}
} // namespace extensions
« no previous file with comments | « no previous file | chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698