Report if video capturing meets output protection requirement.

chrome/browser/media/desktop_capture_access_handler.h

 // Copyright 2015 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_DESKTOP_CAPTURE_ACCESS_HANDLER_H_
 #define CHROME_BROWSER_MEDIA_DESKTOP_CAPTURE_ACCESS_HANDLER_H_
 
 #include <list>
 
 #include "chrome/browser/media/media_access_handler.h"
@@ skipping 17 matching lines @@
   void HandleRequest(content::WebContents* web_contents,
                      const content::MediaStreamRequest& request,
                      const content::MediaResponseCallback& callback,
                      const extensions::Extension* extension) override;
   void UpdateMediaRequestState(int render_process_id,
                                int render_frame_id,
                                int page_request_id,
                                content::MediaStreamType stream_type,
                                content::MediaRequestState state) override;
 
+  // TODO(xjz): To be removed.

[miu, 2016/04/16 00:05:29]

crbug link?

[xjz, 2016/04/21 23:50:44]

Do we need a crbug for this one? It will be removed after content output
protection switches to use the new API.

[miu, 2016/04/26 01:25:13]

That's fine, if you know this will happen soon.

   bool IsCaptureInProgress();
 
+  bool IsCaptureInProgress(int render_process_id,

[miu, 2016/04/16 00:05:29]

Seems like these two new methods should be in the MediaAccessHandler interface.

[xjz, 2016/04/21 23:50:44]

Done. Moved them to the new CaptureAccessHandlerBase class.

+                           int render_frame_id,
+                           bool* is_link_secure);
+  void UpdateCapturingLinkSecured(int render_process_id,
+                                  int render_frame_id,
+                                  int page_request_id,
+                                  content::MediaStreamType stream_type,
+                                  bool is_secure);
+
  private:
   // Tracks MEDIA_DESKTOP_VIDEO_CAPTURE sessions which reach the
   // MEDIA_REQUEST_STATE_DONE state.  Sessions are remove when
   // MEDIA_REQUEST_STATE_CLOSING is encountered.
   struct DesktopCaptureSession {
     int render_process_id;
     int render_frame_id;
     int page_request_id;
+    bool is_capturing_link_secure;
   };
   typedef std::list<DesktopCaptureSession> DesktopCaptureSessions;
 
   void ProcessScreenCaptureAccessRequest(
       content::WebContents* web_contents,
       const content::MediaStreamRequest& request,
       const content::MediaResponseCallback& callback,
       const extensions::Extension* extension);
 
   DesktopCaptureSessions desktop_capture_sessions_;
+
+  // If the extension is authorized for casting.

[miu, 2016/04/16 00:05:29]

nit: Let's expand a bit on this comment here.  How about: "Extensions control
the routing of the captured MediaStream content.  Therefore, links can be deemed
secure only when built-in extensions (and certain whitelisted ones) are in
control of the routing."

[xjz, 2016/04/21 23:50:44]

Moved it into the |session| structure, as it should relate to each capturing
session.

+  bool extension_whitelisted_;

[miu, 2016/04/16 00:05:29]

naming nit (for accuracy): How about: trusted_extension_

[xjz, 2016/04/21 23:50:44]

Moved it into the |Session| structure.

 };
 
 #endif  // CHROME_BROWSER_MEDIA_DESKTOP_CAPTURE_ACCESS_HANDLER_H_