Move ChromeOS login media access logic into a MediaAccessHandler

chrome/browser/media/media_access_handler.cc

+// Copyright 2017 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 "chrome/browser/media/media_access_handler.h"
+
+#include "chrome/browser/media/webrtc/media_capture_devices_dispatcher.h"
+#include "chrome/browser/media/webrtc/media_stream_capture_indicator.h"
+#include "chrome/browser/profiles/profile.h"
+#include "content/public/browser/web_contents.h"
+
+// static
+void MediaAccessHandler::CheckDevicesAndRunCallback(
+    content::WebContents* web_contents,
+    const content::MediaStreamRequest& request,
+    const content::MediaResponseCallback& callback,
+    bool audio_allowed,
+    bool video_allowed) {
+  // TODO(vrk): This code is largely duplicated in
+  // MediaStreamDevicesController::GetDevices(). Move this code into a shared
+  // method
+  // between the two classes.

[emaxx, 2017/04/06 02:41:47]

nit: Reformat the comment.

[raymes, 2017/04/10 00:16:49]

Done.

+  bool get_default_audio_device = audio_allowed;
+  bool get_default_video_device = video_allowed;
+
+  content::MediaStreamDevices devices;
+
+  // Set an initial error result. If neither audio or video is allowed, we'll
+  // never try to get any device below but will just create |ui| and return an
+  // empty list with "invalid state" result. If at least one is allowed, we'll
+  // try to get device(s), and if failure, we want to return "no hardware"
+  // result.
+  // TODO(grunell): The invalid state result should be changed to a new denied
+  // result + a dcheck to ensure at least one of audio or video types is
+  // capture.
+  content::MediaStreamRequestResult result =
+      (audio_allowed || video_allowed) ? content::MEDIA_DEVICE_NO_HARDWARE
+                                       : content::MEDIA_DEVICE_INVALID_STATE;
+
+  // Get the exact audio or video device if an id is specified.
+  // We only set any error result here and before running the callback change
+  // it to OK if we have any device.
+  if (audio_allowed && !request.requested_audio_device_id.empty()) {
+    const content::MediaStreamDevice* audio_device =
+        MediaCaptureDevicesDispatcher::GetInstance()->GetRequestedAudioDevice(
+            request.requested_audio_device_id);
+    if (audio_device) {
+      devices.push_back(*audio_device);
+      get_default_audio_device = false;
+    }
+  }
+  if (video_allowed && !request.requested_video_device_id.empty()) {
+    const content::MediaStreamDevice* video_device =
+        MediaCaptureDevicesDispatcher::GetInstance()->GetRequestedVideoDevice(
+            request.requested_video_device_id);
+    if (video_device) {
+      devices.push_back(*video_device);
+      get_default_video_device = false;
+    }
+  }
+
+  Profile* profile =
+      Profile::FromBrowserContext(web_contents->GetBrowserContext());
+
+  // If either or both audio and video devices were requested but not
+  // specified by id, get the default devices.
+  if (get_default_audio_device || get_default_video_device) {
+    MediaCaptureDevicesDispatcher::GetInstance()->GetDefaultDevicesForProfile(
+        profile, get_default_audio_device, get_default_video_device, &devices);
+  }
+
+  std::unique_ptr<content::MediaStreamUI> ui;
+  if (!devices.empty()) {
+    result = content::MEDIA_DEVICE_OK;
+    ui = MediaCaptureDevicesDispatcher::GetInstance()
+             ->GetMediaStreamCaptureIndicator()
+             ->RegisterMediaStream(web_contents, devices);
+  }
+
+  callback.Run(devices, result, std::move(ui));

[emaxx, 2017/04/06 02:41:47]

nit: #include <utility>

[raymes, 2017/04/10 00:16:49]

Done.

+}