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

Unified Diff: chrome/browser/permissions/permission_queue_controller.cc

Issue 2069343002: Pass profile to PermissionUmaUtil::PermissionIgnored (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@add-hooks-to-permission-layer
Patch Set: Rebase Created 4 years, 5 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/permissions/permission_queue_controller.cc
diff --git a/chrome/browser/permissions/permission_queue_controller.cc b/chrome/browser/permissions/permission_queue_controller.cc
index 5f6bbfc093d27cb91c1f1b5e14d1b34871278040..161e04a10e95ba1f9aa7e04b4566b523cd91aa17 100644
--- a/chrome/browser/permissions/permission_queue_controller.cc
+++ b/chrome/browser/permissions/permission_queue_controller.cc
@@ -52,6 +52,7 @@ class PermissionQueueController::PendingInfobarRequest {
const PermissionRequestID& id,
const GURL& requesting_frame,
const GURL& embedder,
+ Profile* profile,
const PermissionDecidedCallback& callback);
~PendingInfobarRequest();
@@ -71,6 +72,7 @@ class PermissionQueueController::PendingInfobarRequest {
PermissionRequestID id_;
GURL requesting_frame_;
GURL embedder_;
+ Profile* profile_;
PermissionDecidedCallback callback_;
infobars::InfoBar* infobar_;
@@ -82,11 +84,13 @@ PermissionQueueController::PendingInfobarRequest::PendingInfobarRequest(
const PermissionRequestID& id,
const GURL& requesting_frame,
const GURL& embedder,
+ Profile* profile,
const PermissionDecidedCallback& callback)
: type_(type),
id_(id),
requesting_frame_(requesting_frame),
embedder_(embedder),
+ profile_(profile),
callback_(callback),
infobar_(NULL) {}
@@ -119,21 +123,21 @@ void PermissionQueueController::PendingInfobarRequest::CreateInfoBar(
switch (type_) {
case content::PermissionType::GEOLOCATION:
infobar_ = GeolocationInfoBarDelegateAndroid::Create(
- GetInfoBarService(id_), requesting_frame_, callback);
+ GetInfoBarService(id_), requesting_frame_, profile_, callback);
break;
#if defined(ENABLE_NOTIFICATIONS)
case content::PermissionType::NOTIFICATIONS:
infobar_ = NotificationPermissionInfobarDelegate::Create(
- GetInfoBarService(id_), requesting_frame_, callback);
+ GetInfoBarService(id_), requesting_frame_, profile_, callback);
break;
#endif // ENABLE_NOTIFICATIONS
case content::PermissionType::MIDI_SYSEX:
infobar_ = MidiPermissionInfoBarDelegateAndroid::Create(
- GetInfoBarService(id_), requesting_frame_, callback);
+ GetInfoBarService(id_), requesting_frame_, profile_, callback);
break;
case content::PermissionType::PROTECTED_MEDIA_IDENTIFIER:
infobar_ = ProtectedMediaIdentifierInfoBarDelegateAndroid::Create(
- GetInfoBarService(id_), requesting_frame_, callback);
+ GetInfoBarService(id_), requesting_frame_, profile_, callback);
break;
default:
NOTREACHED();
@@ -169,7 +173,7 @@ void PermissionQueueController::CreateInfoBarRequest(
return;
pending_infobar_requests_.push_back(PendingInfobarRequest(
- permission_type_, id, requesting_frame, embedder, callback));
+ permission_type_, id, requesting_frame, embedder, profile_, callback));
if (!AlreadyShowingInfoBarForTab(id))
ShowQueuedInfoBarForTab(id);
}

Powered by Google App Engine
This is Rietveld 408576698