MJPEG acceleration for video capture using VAAPI

content/browser/renderer_host/media/video_capture_buffer_pool.cc

 // Copyright (c) 2013 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.
 
 #include "content/browser/renderer_host/media/video_capture_buffer_pool.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 150 matching lines @@
 }
 
 VideoCaptureBufferPool::Buffer* VideoCaptureBufferPool::GetBuffer(
     int buffer_id) {
   BufferMap::iterator it = buffers_.find(buffer_id);
   if (it == buffers_.end())
     return NULL;
   return it->second;
 }
 
+base::SharedMemoryHandle VideoCaptureBufferPool::GetBufferSharedMemory(
+    int buffer_id) {

[wuchengli, 2015/03/23 06:30:14]

Need to acquire |lock_|.

Or how about merging this with GetBufferInfo and store the handle in
media::VideoCaptureDevice::Client::Buffer?

[mcasas, 2015/03/23 19:29:50]

For another purpose I'm about to start adding a 
VideoCaptureDevice::Client::GetHandle(), which would
be a shared memory fd or a dma-buf fd depending on
the platform capabilities etc. 

Depending on the platform, the BufferPool Buffers might
not be backed by SharedMemory.

[kcwu, 2015/03/30 18:12:14]

Did you mean VideoCaptureDevice::Client::Buffer::GetHandle() ?
Now I'm adding VideoCaptureDevice::Client::Buffer::handle(), which returns
base::SharedMemoryHandle, based on wuchengli's suggestion. What should I do to
fit your plan?

[kcwu, 2015/03/30 18:12:14]

Done.

+  BufferMap::iterator it = buffers_.find(buffer_id);
+  if (it == buffers_.end())
+    return base::SharedMemory::NULLHandle();
+  return it->second->shared_memory.handle();
+}
+
 }  // namespace content