Pass GpuMemoryBuffer backed VideoFrame from browser to renderer processes

content/browser/renderer_host/media/video_capture_host.cc

Index: content/browser/renderer_host/media/video_capture_host.cc
diff --git a/content/browser/renderer_host/media/video_capture_host.cc b/content/browser/renderer_host/media/video_capture_host.cc
index 9d84d92840950f9b934834fd273b027e7afbe998..c5302bf1c21659133283ba0b5b9e22c054e2ecf4 100644
--- a/content/browser/renderer_host/media/video_capture_host.cc
+++ b/content/browser/renderer_host/media/video_capture_host.cc
@@ -66,6 +66,19 @@ void VideoCaptureHost::OnBufferCreated(VideoCaptureControllerID controller_id,
   Send(new VideoCaptureMsg_NewBuffer(controller_id, handle, length, buffer_id));
 }
 
+void VideoCaptureHost::OnBufferCreated2(
+    VideoCaptureControllerID controller_id,
+    const std::vector<gfx::GpuMemoryBufferHandle>& gmb_handles,

[reveman, 2015/08/26 11:54:04]

nit: s/gmb_handles/handles/

[emircan, 2015/08/26 21:23:11]

Done.

+    const gfx::Size& size,
+    int buffer_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  if (entries_.find(controller_id) == entries_.end())

[reveman, 2015/08/26 11:54:04]

can this be a DCHECK?

[emircan, 2015/08/26 21:23:11]

I read through and it all goes down to this CL [0], and it looks like they are
trying to avoid some race conditions. I will try this in a seperate CL and see
how bots react.

[0] https://chromiumcodereview.appspot.com/10537057

[reveman, 2015/08/27 00:21:24]

Sounds good. Thanks.

+    return;
+
+  Send(new VideoCaptureMsg_NewBuffer2(controller_id, gmb_handles, size,
+                                      buffer_id));
+}
+
 void VideoCaptureHost::OnBufferDestroyed(VideoCaptureControllerID controller_id,
                                          int buffer_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);