| Index: chrome/browser/media/webrtc/media_stream_devices_controller.cc
|
| diff --git a/chrome/browser/media/webrtc/media_stream_devices_controller.cc b/chrome/browser/media/webrtc/media_stream_devices_controller.cc
|
| index 9a22945633e6dd771da79a373ab3d53aa0cb4724..ba1a834f22c8f839ec770b58528c02d2e108d1da 100644
|
| --- a/chrome/browser/media/webrtc/media_stream_devices_controller.cc
|
| +++ b/chrome/browser/media/webrtc/media_stream_devices_controller.cc
|
| @@ -58,34 +58,34 @@ namespace {
|
|
|
| // Returns true if the given ContentSettingsType is being requested in
|
| // |request|.
|
| -bool ContentTypeIsRequested(content::PermissionType type,
|
| +bool ContentTypeIsRequested(ContentSettingsType type,
|
| const content::MediaStreamRequest& request) {
|
| if (request.request_type == content::MEDIA_OPEN_DEVICE_PEPPER_ONLY)
|
| return true;
|
|
|
| - if (type == content::PermissionType::AUDIO_CAPTURE)
|
| + if (type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC)
|
| return request.audio_type == content::MEDIA_DEVICE_AUDIO_CAPTURE;
|
|
|
| - if (type == content::PermissionType::VIDEO_CAPTURE)
|
| + if (type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA)
|
| return request.video_type == content::MEDIA_DEVICE_VIDEO_CAPTURE;
|
|
|
| return false;
|
| }
|
|
|
| using PermissionActionCallback =
|
| - base::Callback<void(content::PermissionType,
|
| + base::Callback<void(ContentSettingsType,
|
| PermissionRequestGestureType,
|
| const GURL&,
|
| Profile*)>;
|
|
|
| void RecordSinglePermissionAction(const content::MediaStreamRequest& request,
|
| - content::PermissionType permission_type,
|
| + ContentSettingsType content_type,
|
| Profile* profile,
|
| PermissionActionCallback callback) {
|
| - if (ContentTypeIsRequested(permission_type, request)) {
|
| + if (ContentTypeIsRequested(content_type, request)) {
|
| // TODO(stefanocs): Pass the actual |gesture_type| once this file has been
|
| // refactored into PermissionContext.
|
| - callback.Run(permission_type, PermissionRequestGestureType::UNKNOWN,
|
| + callback.Run(content_type, PermissionRequestGestureType::UNKNOWN,
|
| request.security_origin, profile);
|
| }
|
| }
|
| @@ -94,10 +94,10 @@ void RecordSinglePermissionAction(const content::MediaStreamRequest& request,
|
| void RecordPermissionAction(const content::MediaStreamRequest& request,
|
| Profile* profile,
|
| PermissionActionCallback callback) {
|
| - RecordSinglePermissionAction(request, content::PermissionType::AUDIO_CAPTURE,
|
| - profile, callback);
|
| - RecordSinglePermissionAction(request, content::PermissionType::VIDEO_CAPTURE,
|
| + RecordSinglePermissionAction(request, CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC,
|
| profile, callback);
|
| + RecordSinglePermissionAction(
|
| + request, CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA, profile, callback);
|
| }
|
|
|
| // This helper class helps to measure the number of media stream requests that
|
| @@ -256,16 +256,6 @@ base::string16 MediaStreamDevicesController::GetMessageText() const {
|
| GetOrigin(), url_formatter::SchemeDisplay::OMIT_CRYPTOGRAPHIC));
|
| }
|
|
|
| -content::PermissionType
|
| -MediaStreamDevicesController::GetPermissionTypeForContentSettingsType(
|
| - ContentSettingsType content_type) const {
|
| - DCHECK(content_type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC ||
|
| - content_type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
|
| - content::PermissionType permission = content::PermissionType::NUM;
|
| - CHECK(PermissionUtil::GetPermissionType(content_type, &permission));
|
| - return permission;
|
| -}
|
| -
|
| void MediaStreamDevicesController::ForcePermissionDeniedTemporarily() {
|
| set_persist(false);
|
| // TODO(tsergeant): Determine whether it is appropriate to record permission
|
| @@ -567,9 +557,7 @@ ContentSetting MediaStreamDevicesController::GetContentSetting(
|
| return CONTENT_SETTING_BLOCK;
|
| }
|
|
|
| - content::PermissionType permission_type =
|
| - GetPermissionTypeForContentSettingsType(content_type);
|
| - if (ContentTypeIsRequested(permission_type, request)) {
|
| + if (ContentTypeIsRequested(content_type, request)) {
|
| DCHECK(content::IsOriginSecure(request_.security_origin) ||
|
| request_.request_type == content::MEDIA_OPEN_DEVICE_PEPPER_ONLY);
|
| MediaPermission permission(content_type, request.security_origin,
|
|
|