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

Unified Diff: chrome/browser/media/protected_media_identifier_permission_context.cc

Issue 107413006: Dismiss EME infobar when WebMediaPlayer is destroyed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: make FakeProfile class happy Created 6 years, 11 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: chrome/browser/media/protected_media_identifier_permission_context.cc
diff --git a/chrome/browser/media/protected_media_identifier_permission_context.cc b/chrome/browser/media/protected_media_identifier_permission_context.cc
index e56b41a8c47922fd581032e33827f6ff72a99ba9..604f587170f598f532ce3b35362ac6e13d3a4f01 100644
--- a/chrome/browser/media/protected_media_identifier_permission_context.cc
+++ b/chrome/browser/media/protected_media_identifier_permission_context.cc
@@ -43,6 +43,8 @@ void ProtectedMediaIdentifierPermissionContext::
RequestProtectedMediaIdentifierPermission(
int render_process_id,
int render_view_id,
+ int bridge_id,
+ int group_id,
const GURL& requesting_frame,
const base::Callback<void(bool)>& callback) {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
@@ -51,7 +53,8 @@ void ProtectedMediaIdentifierPermissionContext::
content::WebContents* web_contents =
tab_util::GetWebContentsByID(render_process_id, render_view_id);
- const PermissionRequestID id(render_process_id, render_view_id, 0);
+ const PermissionRequestID id(
+ render_process_id, render_view_id, bridge_id, group_id);
if (extensions::GetViewType(web_contents) !=
extensions::VIEW_TYPE_TAB_CONTENTS) {
@@ -79,12 +82,8 @@ void ProtectedMediaIdentifierPermissionContext::
}
void ProtectedMediaIdentifierPermissionContext::
- CancelProtectedMediaIdentifierPermissionRequest(
- int render_process_id,
- int render_view_id,
- const GURL& requesting_frame) {
- CancelPendingInfoBarRequest(
- PermissionRequestID(render_process_id, render_view_id, 0));
+ CancelProtectedMediaIdentifierPermissionRequests(int group_id) {
+ CancelPendingInfobarRequests(group_id);
}
void ProtectedMediaIdentifierPermissionContext::DecidePermission(
@@ -186,20 +185,20 @@ PermissionQueueController*
}
void
-ProtectedMediaIdentifierPermissionContext::CancelPendingInfoBarRequest(
- const PermissionRequestID& id) {
+ProtectedMediaIdentifierPermissionContext::CancelPendingInfobarRequests(
+ int group_id) {
if (!content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)) {
content::BrowserThread::PostTask(
content::BrowserThread::UI,
FROM_HERE,
base::Bind(&ProtectedMediaIdentifierPermissionContext::
- CancelPendingInfoBarRequest,
+ CancelPendingInfobarRequests,
this,
- id));
+ group_id));
return;
}
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
if (shutting_down_)
return;
- QueueController()->CancelInfoBarRequest(id);
+ QueueController()->CancelInfoBarRequests(group_id);
}

Powered by Google App Engine
This is Rietveld 408576698