media: Make GpuMemoryBuffers VideoFrame copies asynchronous.

media/video/gpu_memory_buffer_video_frame_pool.h

Index: media/video/gpu_memory_buffer_video_frame_pool.h
diff --git a/media/video/gpu_memory_buffer_video_frame_pool.h b/media/video/gpu_memory_buffer_video_frame_pool.h
index e01c791aae9666f84816b660da79210191dcc78a..9effe2f9d082a2d157bd022a71d6177a69b13f58 100644
--- a/media/video/gpu_memory_buffer_video_frame_pool.h
+++ b/media/video/gpu_memory_buffer_video_frame_pool.h
@@ -7,9 +7,11 @@
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "base/task_runner.h"
 #include "media/base/video_frame.h"
 
 namespace base {
+class SequencedTaskRunner;

[reveman, 2015/08/17 16:38:29]

needed?

[Daniele Castagna, 2015/08/19 21:31:25]

Done.

 class SingleThreadTaskRunner;
 }
 
@@ -26,17 +28,24 @@ class GpuVideoAcceleratorFactories;
 class MEDIA_EXPORT GpuMemoryBufferVideoFramePool {
  public:
   GpuMemoryBufferVideoFramePool(
-      const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
+      const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
+      const scoped_refptr<base::TaskRunner>& worker_task_runner,
       const scoped_refptr<GpuVideoAcceleratorFactories>& gpu_factories);
   ~GpuMemoryBufferVideoFramePool();
 
-  // Returns a new VideoFrame containing only mailboxes to native resources.
-  // The content of the returned object is copied from the software-allocated
+  // Callback used by MaybeCreateHardwareFrame to deliver a new VideoFrame
+  // after it has been copied to GpuMemoryBuffers.
+  typedef base::Callback<void(const scoped_refptr<VideoFrame>&)> FrameReadyCB;
+
+  // Calls |cb| on |media_worker_pool| with a new VideoFrame containing only
+  // mailboxes to native resources. |cb| will be destroyed on
+  // |media_worker_pool|.
+  // The content of the new object is copied from the software-allocated
   // |video_frame|.
   // If it's not possible to create a new hardware VideoFrame, |video_frame|
-  // itself will be returned.
-  scoped_refptr<VideoFrame> MaybeCreateHardwareFrame(
-      const scoped_refptr<VideoFrame>& video_frame);
+  // itself will passed to |cb|.
+  void MaybeCreateHardwareFrame(const scoped_refptr<VideoFrame>& video_frame,
+                                const FrameReadyCB& cb);
 
  private:
   class PoolImpl;