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

Unified Diff: content/browser/permissions/permission_service_impl.h

Issue 1419083002: Permissions: browser/ side implementation of multiple permissions request. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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
Index: content/browser/permissions/permission_service_impl.h
diff --git a/content/browser/permissions/permission_service_impl.h b/content/browser/permissions/permission_service_impl.h
index 26321870d3d7bcd6d210410a76496d5245f4742a..5d5351193fa5c79cbbbfacb5647b77931df1cff5 100644
--- a/content/browser/permissions/permission_service_impl.h
+++ b/content/browser/permissions/permission_service_impl.h
@@ -44,15 +44,14 @@ class PermissionServiceImpl : public PermissionService {
mojo::Callback<void(mojo::Array<PermissionStatus>)>;
struct PendingRequest {
- PendingRequest(PermissionType permission, const GURL& origin,
- const PermissionStatusCallback& callback);
+ PendingRequest(const PermissionsStatusCallback& callback,
+ int request_count);
~PendingRequest();
// Request ID received from the PermissionManager.
int id;
- PermissionType permission;
- GURL origin;
- PermissionStatusCallback callback;
+ PermissionsStatusCallback callback;
+ int request_count;
};
using RequestsMap = IDMap<PendingRequest, IDMapOwnPointer>;
@@ -92,7 +91,12 @@ class PermissionServiceImpl : public PermissionService {
void OnConnectionError();
- void OnRequestPermissionResponse(int request_id, PermissionStatus status);
+ void OnRequestPermissionResponse(
+ int pending_request_id,
+ PermissionStatus status);
+ void OnRequestPermissionsResponse(
+ int pending_request_id,
+ const std::vector<PermissionStatus>& result);
PermissionStatus GetPermissionStatusFromName(PermissionName permission,
const GURL& origin);
« no previous file with comments | « chromecast/browser/cast_permission_manager.cc ('k') | content/browser/permissions/permission_service_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698