MJPEG acceleration for video capture using VAAPI

content/browser/renderer_host/media/video_capture_buffer_pool.h

Index: content/browser/renderer_host/media/video_capture_buffer_pool.h
diff --git a/content/browser/renderer_host/media/video_capture_buffer_pool.h b/content/browser/renderer_host/media/video_capture_buffer_pool.h
index 25464997c6e4b523c5d5fce46110d153966edfd2..05b8865e8a63ae373b66aab3a185588a7aaac487 100644
--- a/content/browser/renderer_host/media/video_capture_buffer_pool.h
+++ b/content/browser/renderer_host/media/video_capture_buffer_pool.h
@@ -55,7 +55,10 @@ class CONTENT_EXPORT VideoCaptureBufferPool
 
   // Query the memory parameters of |buffer_id|. Fills in parameters in the
   // pointer arguments, and returns true iff the buffer exists.
-  bool GetBufferInfo(int buffer_id, void** storage, size_t* size);
+  bool GetBufferInfo(int buffer_id,
+                     void** storage,
+                     size_t* size,
+                     base::SharedMemoryHandle* handle);

[mcasas, 2015/04/28 00:04:50]

You'll have to rebase this area quite a bit. Basically extending
BufferHandle [1] and implementing it in SimpleBufferHandle [2].
I'd like to suggest AsPlatformHandle() as the new virtual
method name, then GpuMemoryBufferBufferHandle [3]
would implement it as gmb_->GetHandle();

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/re...
[2]
https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/re...
[3]
https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/re...

[kcwu, 2015/04/30 19:25:43]

Done.
Thanks for your kind help. Please take a look.
I'm not sure should they return "gfx::GpuMemoryBufferHandle" everywhere or not.

 
   // Reserve or allocate a buffer to support a packed frame of |dimensions| of
   // pixel |format| and return its id. This will fail (returning kInvalidId) if
@@ -114,6 +117,7 @@ class CONTENT_EXPORT VideoCaptureBufferPool
     virtual size_t requested_size() = 0;
     // The actual size of the underlying backing resource.
     virtual size_t mapped_size() = 0;
+    virtual base::SharedMemoryHandle handle() = 0;
 
     virtual bool ShareToProcess(base::ProcessHandle process_handle,
                                 base::SharedMemoryHandle* new_handle) = 0;