Move ChromeOS login media access logic into a MediaAccessHandler

chrome/browser/media/saml_media_access_handler.h

Index: chrome/browser/media/saml_media_access_handler.h
diff --git a/chrome/browser/media/extension_media_access_handler.h b/chrome/browser/media/saml_media_access_handler.h
similarity index 56%
copy from chrome/browser/media/extension_media_access_handler.h
copy to chrome/browser/media/saml_media_access_handler.h
index fe9908b59c48e471913d80d5290f7caa62485fa0..bf12385a22c7dfae535c042941dc340cad0d96da 100644
--- a/chrome/browser/media/extension_media_access_handler.h
+++ b/chrome/browser/media/saml_media_access_handler.h
@@ -1,20 +1,22 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
+// Copyright 2017 The Chromium Authors. All rights reserved.

[achuithb, 2017/03/16 03:46:04]

Don't think this should be changed.

[raymes, 2017/04/03 03:30:11]

This is a new file being added so it's appropriate to update the date.

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_MEDIA_EXTENSION_MEDIA_ACCESS_HANDLER_H_
-#define CHROME_BROWSER_MEDIA_EXTENSION_MEDIA_ACCESS_HANDLER_H_
+#ifndef CHROME_BROWSER_MEDIA_SAML_MEDIA_ACCESS_HANDLER_H_

[achuithb, 2017/03/16 03:46:04]

Should this file be moved to chrome/browser/chromeos?

[raymes, 2017/04/03 03:30:11]

I can move it, but all the other media access handlers are here, including
extensions ones. I defer to sergeyu@ on that.

+#define CHROME_BROWSER_MEDIA_SAML_MEDIA_ACCESS_HANDLER_H_
 
 #include "chrome/browser/media/media_access_handler.h"
 
-// MediaAccessHandler for extension capturing requests.
-class ExtensionMediaAccessHandler : public MediaAccessHandler {
+// MediaAccessHandler for media requests coming from SAML login pages on
+// ChromeOS.
+class SamlMediaAccessHandler : public MediaAccessHandler {
  public:
-  ExtensionMediaAccessHandler();
-  ~ExtensionMediaAccessHandler() override;
+  SamlMediaAccessHandler();
+  ~SamlMediaAccessHandler() override;
 
   // MediaAccessHandler implementation.
-  bool SupportsStreamType(const content::MediaStreamType type,
+  bool SupportsStreamType(content::WebContents* web_contents,
+                          const content::MediaStreamType type,
                           const extensions::Extension* extension) override;
   bool CheckMediaAccessPermission(
       content::WebContents* web_contents,
@@ -27,4 +29,4 @@ class ExtensionMediaAccessHandler : public MediaAccessHandler {
                      const extensions::Extension* extension) override;
 };
 
-#endif  // CHROME_BROWSER_MEDIA_EXTENSION_MEDIA_ACCESS_HANDLER_H_
+#endif  // CHROME_BROWSER_MEDIA_SAML_MEDIA_ACCESS_HANDLER_H_