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

Unified Diff: extensions/common/permissions/api_permission_set.cc

Issue 2697833004: Revert of Public Sessions - prompt the user for pageCapture requests (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « extensions/common/permissions/api_permission_set.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/common/permissions/api_permission_set.cc
diff --git a/extensions/common/permissions/api_permission_set.cc b/extensions/common/permissions/api_permission_set.cc
index a6a11f56350a67e5b05e0a7c059a923024db5ad0..d5b14a083ef52db2477e5d31b3781f411359071f 100644
--- a/extensions/common/permissions/api_permission_set.cc
+++ b/extensions/common/permissions/api_permission_set.cc
@@ -197,13 +197,6 @@
PermissionIDSet::PermissionIDSet() {
}
-PermissionIDSet::PermissionIDSet(
- std::initializer_list<APIPermission::ID> permissions) {
- for (auto permission : permissions) {
- permissions_.insert(PermissionID(permission));
- }
-}
-
PermissionIDSet::PermissionIDSet(const PermissionIDSet& other) = default;
PermissionIDSet::~PermissionIDSet() {
@@ -243,13 +236,9 @@
return params;
}
-bool PermissionIDSet::ContainsID(PermissionID permission_id) const {
- auto it = permissions_.lower_bound(permission_id);
- return it != permissions_.end() && it->id() == permission_id.id();
-}
-
bool PermissionIDSet::ContainsID(APIPermission::ID permission_id) const {
- return ContainsID(PermissionID(permission_id));
+ auto it = permissions_.lower_bound(PermissionID(permission_id));
+ return it != permissions_.end() && it->id() == permission_id;
}
bool PermissionIDSet::ContainsAllIDs(
@@ -271,14 +260,6 @@
return false;
}
-bool PermissionIDSet::ContainsAnyID(const PermissionIDSet& other) const {
- for (const auto& id : other) {
- if (ContainsID(id))
- return true;
- }
- return false;
-}
-
PermissionIDSet PermissionIDSet::GetAllPermissionsWithID(
APIPermission::ID permission_id) const {
PermissionIDSet subset;
« no previous file with comments | « extensions/common/permissions/api_permission_set.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698