Move ChromeOS login media access logic into a MediaAccessHandler

chrome/browser/media/saml_media_access_handler.h

-// 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,
       const GURL& security_origin,
       content::MediaStreamType type,
       const extensions::Extension* extension) override;
   void HandleRequest(content::WebContents* web_contents,
                      const content::MediaStreamRequest& request,
                      const content::MediaResponseCallback& callback,
                      const extensions::Extension* extension) override;
 };
 
-#endif  // CHROME_BROWSER_MEDIA_EXTENSION_MEDIA_ACCESS_HANDLER_H_
+#endif  // CHROME_BROWSER_MEDIA_SAML_MEDIA_ACCESS_HANDLER_H_