Chromium Code Reviews| 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 62144e7c308eaf60e18acc8b73213dcef83ba819..e5782e3c1190016f8dc3347976ecf240d5cd3627 100644 |
| --- a/chrome/browser/media/media_stream_devices_controller.cc |
| +++ b/chrome/browser/media/media_stream_devices_controller.cc |
| @@ -71,15 +71,20 @@ bool ContentTypeIsRequested(content::PermissionType type, |
| return false; |
| } |
| -using PermissionActionCallback = |
| - base::Callback<void(content::PermissionType, const GURL&, Profile*)>; |
| +using PermissionActionCallback = base::Callback<void(content::PermissionType, |
| + bool /* user_gesture */, |
|
Sergey Ulanov
2016/07/21 19:32:47
nit: don't need to have the name commented (and ca
stefanocs
2016/07/22 00:56:11
Done.
|
| + const GURL&, |
| + 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); |
| + // TODO(stefanocs): Pass the actual |user_gesture| once this file has been |
| + // refactored into PermissionContext. |
| + callback.Run(permission_type, false /* user_gesture */, |
| + request.security_origin, profile); |
| } |
| } |