Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "content/renderer/media/media_permission_dispatcher.h" | 5 #include "content/renderer/media/media_permission_dispatcher.h" |
| 6 | 6 |
| 7 #include "base/bind.h" | 7 #include "base/bind.h" |
| 8 #include "base/logging.h" | 8 #include "base/logging.h" |
| 9 #include "base/single_thread_task_runner.h" | 9 #include "base/single_thread_task_runner.h" |
| 10 #include "base/thread_task_runner_handle.h" | 10 #include "base/thread_task_runner_handle.h" |
| 11 #include "media/base/bind_to_current_loop.h" | 11 #include "media/base/bind_to_current_loop.h" |
| 12 #include "url/gurl.h" | 12 #include "url/gurl.h" |
| 13 | 13 |
| 14 namespace { | 14 namespace { |
| 15 | 15 |
| 16 using Type = media::MediaPermission::Type; | 16 using Type = media::MediaPermission::Type; |
| 17 | 17 |
| 18 blink::mojom::PermissionName MediaPermissionTypeToPermissionName(Type type) { | 18 permissions::mojom::PermissionName MediaPermissionTypeToPermissionName( |
| 19 Type type) { | |
| 19 switch (type) { | 20 switch (type) { |
| 20 case Type::PROTECTED_MEDIA_IDENTIFIER: | 21 case Type::PROTECTED_MEDIA_IDENTIFIER: |
| 21 return blink::mojom::PermissionName::PROTECTED_MEDIA_IDENTIFIER; | 22 return permissions::mojom::PermissionName::PROTECTED_MEDIA_IDENTIFIER; |
| 22 case Type::AUDIO_CAPTURE: | 23 case Type::AUDIO_CAPTURE: |
| 23 return blink::mojom::PermissionName::AUDIO_CAPTURE; | 24 return permissions::mojom::PermissionName::AUDIO_CAPTURE; |
| 24 case Type::VIDEO_CAPTURE: | 25 case Type::VIDEO_CAPTURE: |
| 25 return blink::mojom::PermissionName::VIDEO_CAPTURE; | 26 return permissions::mojom::PermissionName::VIDEO_CAPTURE; |
| 26 } | 27 } |
| 27 NOTREACHED(); | 28 NOTREACHED(); |
| 28 return blink::mojom::PermissionName::PROTECTED_MEDIA_IDENTIFIER; | 29 return permissions::mojom::PermissionName::PROTECTED_MEDIA_IDENTIFIER; |
| 29 } | 30 } |
| 30 | 31 |
| 31 } // namespace | 32 } // namespace |
| 32 | 33 |
| 33 namespace content { | 34 namespace content { |
| 34 | 35 |
| 35 MediaPermissionDispatcher::MediaPermissionDispatcher( | 36 MediaPermissionDispatcher::MediaPermissionDispatcher( |
| 36 const ConnectToServiceCB& connect_to_service_cb) | 37 const ConnectToServiceCB& connect_to_service_cb) |
| 37 : connect_to_service_cb_(connect_to_service_cb), | 38 : connect_to_service_cb_(connect_to_service_cb), |
| 38 task_runner_(base::ThreadTaskRunnerHandle::Get()), | 39 task_runner_(base::ThreadTaskRunnerHandle::Get()), |
| (...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 108 | 109 |
| 109 uint32_t request_id = next_request_id_++; | 110 uint32_t request_id = next_request_id_++; |
| 110 DCHECK(!requests_.count(request_id)); | 111 DCHECK(!requests_.count(request_id)); |
| 111 requests_[request_id] = permission_status_cb; | 112 requests_[request_id] = permission_status_cb; |
| 112 | 113 |
| 113 return request_id; | 114 return request_id; |
| 114 } | 115 } |
| 115 | 116 |
| 116 void MediaPermissionDispatcher::OnPermissionStatus( | 117 void MediaPermissionDispatcher::OnPermissionStatus( |
| 117 uint32_t request_id, | 118 uint32_t request_id, |
| 118 blink::mojom::PermissionStatus status) { | 119 permissions::mojom::PermissionStatus status) { |
| 119 DVLOG(2) << __FUNCTION__ << ": (" << request_id << ", " << status << ")"; | 120 DVLOG(2) << __FUNCTION__ << ": (" << request_id << ", " << status << ")"; |
| 120 DCHECK(task_runner_->RunsTasksOnCurrentThread()); | 121 DCHECK(task_runner_->RunsTasksOnCurrentThread()); |
| 121 | 122 |
| 122 RequestMap::iterator iter = requests_.find(request_id); | 123 RequestMap::iterator iter = requests_.find(request_id); |
| 123 DCHECK(iter != requests_.end()) << "Request not found."; | 124 DCHECK(iter != requests_.end()) << "Request not found."; |
| 124 | 125 |
| 125 PermissionStatusCB permission_status_cb = iter->second; | 126 PermissionStatusCB permission_status_cb = iter->second; |
| 126 requests_.erase(iter); | 127 requests_.erase(iter); |
| 127 | 128 |
| 128 permission_status_cb.Run(status == blink::mojom::PermissionStatus::GRANTED); | 129 permission_status_cb.Run(status == |
| 130 permissions::mojom::PermissionStatus::GRANTED); | |
|
dcheng
2016/05/02 18:28:53
Huh, is this clang-format? I'm surprised it didn't
xhwang
2016/05/02 18:35:31
Apparently yes :( I've stopped manual formatting f
dcheng
2016/05/02 21:30:46
I've filed a bug upstream: https://llvm.org/bugs/s
| |
| 129 } | 131 } |
| 130 | 132 |
| 131 } // namespace content | 133 } // namespace content |
| OLD | NEW |