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

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

Issue 2075523002: Add SourceUI field to permission report (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 6 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/media_stream_devices_controller.cc
diff --git a/chrome/browser/media/media_stream_devices_controller.cc b/chrome/browser/media/media_stream_devices_controller.cc
index 8995a288f6f048a886b8ebdd4f77ae328890427f..040a68f5cb964cf584e0dd0503940278640475fb 100644
--- a/chrome/browser/media/media_stream_devices_controller.cc
+++ b/chrome/browser/media/media_stream_devices_controller.cc
@@ -71,15 +71,15 @@ bool ContentTypeIsRequested(content::PermissionType type,
return false;
}
-using PermissionActionCallback =
- base::Callback<void(content::PermissionType, const GURL&, Profile*)>;
+using PermissionActionCallback = base::Callback<
kcarattini 2016/07/06 06:54:08 Make sure you coordinate with Liu who is refactori
stefanocs 2016/07/07 01:26:03 Done. She is okay with this.
+ void(content::PermissionType, const GURL&, SourceUI, Profile*)>;
void RecordSinglePermissionAction(const content::MediaStreamRequest& request,
content::PermissionType permission_type,
Profile* profile,
PermissionActionCallback callback) {
if (ContentTypeIsRequested(permission_type, request)) {
- callback.Run(permission_type, request.security_origin, profile);
+ callback.Run(permission_type, request.security_origin, PROMPT, profile);
}
}

Powered by Google App Engine
This is Rietveld 408576698