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.
 // 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_
+#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

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

I have no idea what "SAML" is. Is it "Security Assertion Markup Language" or
something else? It should be clear from the comments.

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

Me neither actually :) +achuith to clarify.

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

Yes, it's a scheme for federated identity providers. This is useful for
enterprises who want to use their own identity providers.

+// 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_