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

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

Issue 1342833002: permissions: handle request ids for permissions in permission manager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 5 years, 3 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.cc
diff --git a/content/browser/permissions/permission_service_impl.cc b/content/browser/permissions/permission_service_impl.cc
index 23bf0981c4b12940354476927567553a8690b584..13a8396af97b95ee7941c1dd1be8c74a6bf2b225 100644
--- a/content/browser/permissions/permission_service_impl.cc
+++ b/content/browser/permissions/permission_service_impl.cc
@@ -45,7 +45,8 @@ PermissionServiceImpl::PendingRequest::PendingRequest(
PermissionType permission,
const GURL& origin,
const PermissionStatusCallback& callback)
- : permission(permission),
+ : id(-1),
mlamouri (slow - plz ping) 2015/09/23 14:46:00 I think you could use the constant here.
Lalit Maganti 2015/09/23 15:24:30 Done.
+ permission(permission),
origin(origin),
callback(callback) {
}
@@ -117,18 +118,25 @@ void PermissionServiceImpl::RequestPermission(
}
PermissionType permission_type = PermissionNameToPermissionType(permission);
- int request_id = pending_requests_.Add(
+ int pending_request_id = pending_requests_.Add(
new PendingRequest(permission_type, GURL(origin), callback));
- browser_context->GetPermissionManager()->RequestPermission(
+ int id = browser_context->GetPermissionManager()->RequestPermission(
permission_type,
context_->render_frame_host(),
- request_id,
GURL(origin),
user_gesture, // TODO(mlamouri): should be removed (crbug.com/423770)
base::Bind(&PermissionServiceImpl::OnRequestPermissionResponse,
weak_factory_.GetWeakPtr(),
- request_id));
+ pending_request_id));
+
+ // Check if the request still exists. It may have been removed by the
+ // the response callback.
mlamouri (slow - plz ping) 2015/09/23 14:46:00 nit: could you rephrase to explain that the callba
Lalit Maganti 2015/09/23 15:24:30 Done.
+ PendingRequest* pending_request = pending_requests_.Lookup(
+ pending_request_id);
+ if (!pending_request)
+ return;
+ pending_request->id = id;
}
void PermissionServiceImpl::RequestPermissions(
@@ -158,7 +166,6 @@ void PermissionServiceImpl::OnRequestPermissionResponse(
}
void PermissionServiceImpl::CancelPendingOperations() {
- DCHECK(context_->render_frame_host());
DCHECK(context_->GetBrowserContext());
PermissionManager* permission_manager =
@@ -170,10 +177,7 @@ void PermissionServiceImpl::CancelPendingOperations() {
for (RequestsMap::Iterator<PendingRequest> it(&pending_requests_);
!it.IsAtEnd(); it.Advance()) {
permission_manager->CancelPermissionRequest(
- it.GetCurrentValue()->permission,
- context_->render_frame_host(),
- it.GetCurrentKey(),
- it.GetCurrentValue()->origin);
+ it.GetCurrentValue()->id);
}
pending_requests_.Clear();

Powered by Google App Engine
This is Rietveld 408576698