Introduce MediaDevicesPermissionChecker.

content/browser/media/media_devices_permission_checker.cc

Index: content/browser/media/media_devices_permission_checker.cc
diff --git a/content/browser/media/media_devices_permission_checker.cc b/content/browser/media/media_devices_permission_checker.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6c0c8e0a26fb58773c6057a92abd363eeb4d8468
--- /dev/null
+++ b/content/browser/media/media_devices_permission_checker.cc
@@ -0,0 +1,154 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/browser/media/media_devices_permission_checker.h"
+
+#include <utility>
+#include <vector>
+
+#include "base/bind.h"
+#include "base/command_line.h"
+#include "content/browser/frame_host/render_frame_host_delegate.h"
+#include "content/browser/frame_host/render_frame_host_impl.h"
+#include "content/common/media/media_devices.h"
+#include "content/public/browser/browser_context.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/common/content_switches.h"
+#include "url/gurl.h"
+#include "url/origin.h"
+
+namespace content {
+
+namespace {
+
+MediaDevicesManager::BoolDeviceTypes CheckPermissionsOnUIThread(
+    MediaDevicesManager::BoolDeviceTypes requested_device_types,
+    int render_process_id,
+    int routing_id,
+    const url::Origin& security_origin) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  RenderFrameHostImpl* frame_host =
+      RenderFrameHostImpl::FromID(render_process_id, routing_id);
+
+  // If there is no |frame_host|, return false for all permissions.
+  if (!frame_host)
+    return MediaDevicesManager::BoolDeviceTypes();
+
+  RenderFrameHostDelegate* delegate = frame_host->delegate();
+  GURL origin = security_origin.GetURL();
+
+  // Currently, the MEDIA_DEVICE_AUDIO_CAPTURE permission is used for
+  // both audio input and output.
+  // TODO(guidou): use specific permission for audio output when it becomes
+  // available. See http://crbug.com/556542.
+  bool has_audio_permission =
+      (requested_device_types[MEDIA_DEVICE_TYPE_AUDIO_INPUT] ||
+       requested_device_types[MEDIA_DEVICE_TYPE_AUDIO_OUTPUT]) &&
+      delegate->CheckMediaAccessPermission(origin, MEDIA_DEVICE_AUDIO_CAPTURE);
+
+  MediaDevicesManager::BoolDeviceTypes result;
+  result[MEDIA_DEVICE_TYPE_AUDIO_INPUT] = has_audio_permission;
+  result[MEDIA_DEVICE_TYPE_AUDIO_OUTPUT] = has_audio_permission;
+  result[MEDIA_DEVICE_TYPE_VIDEO_INPUT] =
+      requested_device_types[MEDIA_DEVICE_TYPE_VIDEO_INPUT] &&
+      delegate->CheckMediaAccessPermission(origin, MEDIA_DEVICE_VIDEO_CAPTURE);
+
+  return result;
+}
+
+bool CheckSinglePermissionOnUIThread(MediaDeviceType device_type,
+                                     int render_process_id,
+                                     int routing_id,
+                                     const url::Origin& security_origin) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  MediaDevicesManager::BoolDeviceTypes requested;
+  requested[device_type] = true;
+  MediaDevicesManager::BoolDeviceTypes result = CheckPermissionsOnUIThread(
+      requested, render_process_id, routing_id, security_origin);
+  return result[device_type];
+}
+
+}  // namespace
+
+MediaDevicesPermissionChecker::MediaDevicesPermissionChecker()
+    : use_override_(base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kUseFakeUIForMediaStream)),
+      override_value_(
+          base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+              switches::kUseFakeUIForMediaStream) != "deny") {}
+
+bool MediaDevicesPermissionChecker::CheckPermission(
+    MediaDeviceType device_type,
+    int render_process_id,
+    int routing_id,
+    const url::Origin& security_origin) {
+  if (use_override_)
+    return override_value_;
+
+  return CheckSinglePermissionOnUIThread(device_type, render_process_id,
+                                         routing_id, security_origin);
+}
+
+void MediaDevicesPermissionChecker::CheckPermission(
+    MediaDeviceType device_type,
+    int render_process_id,
+    int routing_id,
+    const url::Origin& security_origin,
+    const base::Callback<void(bool)>& callback) {
+  if (use_override_) {
+    callback.Run(override_value_);
+    return;
+  }
+
+  BrowserThread::PostTaskAndReplyWithResult(
+      BrowserThread::UI, FROM_HERE,
+      base::Bind(&CheckSinglePermissionOnUIThread, device_type,
+                 render_process_id, routing_id, security_origin),
+      callback);
+}
+
+MediaDevicesManager::BoolDeviceTypes
+MediaDevicesPermissionChecker::CheckPermissions(
+    MediaDevicesManager::BoolDeviceTypes requested_device_types,
+    int render_process_id,
+    int routing_id,
+    const url::Origin& security_origin) {
+  if (use_override_) {
+    MediaDevicesManager::BoolDeviceTypes result;
+    result.fill(override_value_);
+    return result;
+  }
+
+  return CheckPermissionsOnUIThread(requested_device_types, render_process_id,
+                                    routing_id, security_origin);
+}
+
+void MediaDevicesPermissionChecker::CheckPermissions(
+    MediaDevicesManager::BoolDeviceTypes requested,
+    int render_process_id,
+    int routing_id,
+    const url::Origin& security_origin,
+    const base::Callback<void(const MediaDevicesManager::BoolDeviceTypes&)>&
+        callback) {
+  if (use_override_) {
+    MediaDevicesManager::BoolDeviceTypes result;
+    result.fill(override_value_);
+    callback.Run(result);
+    return;
+  }
+
+  BrowserThread::PostTaskAndReplyWithResult(
+      BrowserThread::UI, FROM_HERE,
+      base::Bind(&CheckPermissionsOnUIThread, requested, render_process_id,
+                 routing_id, security_origin),
+      callback);
+}
+
+void MediaDevicesPermissionChecker::OverridePermissions(bool override_value) {
+  use_override_ = true;

[tommi (sloooow) - chröme, 2016/10/28 09:36:52]

nit: would it make sense to use Optional?

[Guido Urdaneta, 2016/10/28 10:06:55]

I would work, but I think the code would become less obvious and more error
prone since both |override| and |*override| would be (implicitly) bool.

+  override_value_ = override_value;
+}
+
+}  // namespace content