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

Unified Diff: chrome/common/extensions/permissions/permission_set.cc

Issue 11348335: Revert 170514 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 1 month 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
Index: chrome/common/extensions/permissions/permission_set.cc
===================================================================
--- chrome/common/extensions/permissions/permission_set.cc (revision 170599)
+++ chrome/common/extensions/permissions/permission_set.cc (working copy)
@@ -197,23 +197,6 @@
return new PermissionSet(apis, explicit_hosts, scriptable_hosts);
}
-// static
-PermissionSet* PermissionSet::ExcludeNotInManifestPermissions(
- const PermissionSet* set) {
- if (!set)
- return new PermissionSet();
-
- APIPermissionSet apis;
- for (APIPermissionSet::const_iterator i = set->apis().begin();
- i != set->apis().end(); ++i) {
- if (!i->ManifestEntryForbidden())
- apis.insert(i->Clone());
- }
-
- return new PermissionSet(
- apis, set->explicit_hosts(), set->scriptable_hosts());
-}
-
bool PermissionSet::operator==(
const PermissionSet& rhs) const {
return apis_ == rhs.apis_ &&
« no previous file with comments | « chrome/common/extensions/permissions/permission_set.h ('k') | chrome/common/extensions/permissions/permission_set_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698