Move ChromeOS login media access logic into a MediaAccessHandler

chrome/browser/media/media_access_handler.cc

-// Copyright 2015 The Chromium Authors. All rights reserved.
+// 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/extension_media_access_handler.h"
-
-#include <utility>
+#include "chrome/browser/media/saml_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/media/webrtc/media_stream_device_permissions.h"
 #include "chrome/browser/profiles/profile.h"
-#include "chrome/common/pref_names.h"
 #include "content/public/browser/web_contents.h"
-#include "extensions/common/extension.h"
-#include "extensions/common/permissions/permissions_data.h"
 
-namespace {
-
-// This is a short-term solution to grant camera and/or microphone access to
-// extensions:
-// 1. Virtual keyboard extension.
-// 2. Flutter gesture recognition extension.
-// 3. TODO(smus): Airbender experiment 1.
-// 4. TODO(smus): Airbender experiment 2.
-// 5. Hotwording component extension.
-// 6. XKB input method component extension.
-// 7. M17n/T13n/CJK input method component extension.
-// Once http://crbug.com/292856 is fixed, remove this whitelist.
-bool IsMediaRequestWhitelistedForExtension(
-    const extensions::Extension* extension) {
-  return extension->id() == "mppnpdlheglhdfmldimlhpnegondlapf" ||
-         extension->id() == "jokbpnebhdcladagohdnfgjcpejggllo" ||
-         extension->id() == "clffjmdilanldobdnedchkdbofoimcgb" ||
-         extension->id() == "nnckehldicaciogcbchegobnafnjkcne" ||
-         extension->id() == "nbpagnldghgfoolbancepceaanlmhfmd" ||
-         extension->id() == "jkghodnilhceideoidjikpgommlajknk" ||
-         extension->id() == "gjaehgfemfahhmlgpdfknkhdnemmolop";
-}
-
-}  // namespace
-
-ExtensionMediaAccessHandler::ExtensionMediaAccessHandler() {
-}
-
-ExtensionMediaAccessHandler::~ExtensionMediaAccessHandler() {
-}
-
-bool ExtensionMediaAccessHandler::SupportsStreamType(
-    const content::MediaStreamType type,
-    const extensions::Extension* extension) {
-  return extension && (extension->is_platform_app() ||
-                       IsMediaRequestWhitelistedForExtension(extension)) &&
-         (type == content::MEDIA_DEVICE_AUDIO_CAPTURE ||
-          type == content::MEDIA_DEVICE_VIDEO_CAPTURE);
-}
-
-bool ExtensionMediaAccessHandler::CheckMediaAccessPermission(
-    content::WebContents* web_contents,
-    const GURL& security_origin,
-    content::MediaStreamType type,
-    const extensions::Extension* extension) {
-  return extension->permissions_data()->HasAPIPermission(
-      type == content::MEDIA_DEVICE_AUDIO_CAPTURE
-          ? extensions::APIPermission::kAudioCapture
-          : extensions::APIPermission::kVideoCapture);
-}
-
-void ExtensionMediaAccessHandler::HandleRequest(
+// static
+void MediaAccessHandler::CheckDevicesAndRunCallback(
     content::WebContents* web_contents,
     const content::MediaStreamRequest& request,
     const content::MediaResponseCallback& callback,
-    const extensions::Extension* extension) {
+    bool audio_allowed,
+    bool video_allowed) {
   // TODO(vrk): This code is largely duplicated in
   // MediaStreamDevicesController::Accept(). Move this code into a shared method

[Sergey Ulanov, 2017/03/15 22:48:30]

That code is in MediaStreamDevicesController::GetDevices() now. Update this TODO
please

[raymes, 2017/03/16 02:30:37]

Done.

   // between the two classes.
-
-  Profile* profile =
-      Profile::FromBrowserContext(web_contents->GetBrowserContext());
-
-  bool audio_allowed =
-      request.audio_type == content::MEDIA_DEVICE_AUDIO_CAPTURE &&
-      extension->permissions_data()->HasAPIPermission(
-          extensions::APIPermission::kAudioCapture) &&
-      GetDevicePolicy(profile, extension->url(), prefs::kAudioCaptureAllowed,
-                      prefs::kAudioCaptureAllowedUrls) != ALWAYS_DENY;
-  bool video_allowed =
-      request.video_type == content::MEDIA_DEVICE_VIDEO_CAPTURE &&
-      extension->permissions_data()->HasAPIPermission(
-          extensions::APIPermission::kVideoCapture) &&
-      GetDevicePolicy(profile, extension->url(), prefs::kVideoCaptureAllowed,
-                      prefs::kVideoCaptureAllowedUrls) != ALWAYS_DENY;
-
   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.
@@ skipping 19 matching lines @@
   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));
 }