MJPEG acceleration for video capture using VAAPI

content/browser/renderer_host/media/video_capture_buffer_pool.h

 // 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_BUFFER_POOL_H_
 #define CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_BUFFER_POOL_H_
 
 #include <map>
 
 #include "base/basictypes.h"
@@ skipping 37 matching lines @@
 
   // One-time (per client/per-buffer) initialization to share a particular
   // buffer to a process. The size of the allocation is returned as
   // |memory_size|.
   base::SharedMemoryHandle ShareToProcess(int buffer_id,
                                           base::ProcessHandle process_handle,
                                           size_t* memory_size);
 
   // 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
   // the pool already is at its |count| limit of the number of allocations, and
   // all allocated buffers are in use by the producer and/or consumers.
   //
   // If successful, the reserved buffer remains reserved (and writable by the
   // producer) until ownership is transferred either to the consumer via
   // HoldForConsumers(), or back to the pool with
   // RelinquishProducerReservation().
@@ skipping 38 matching lines @@
     void set_consumer_hold_count(int value) { consumer_hold_count_ = value; }
 
     // Returns a void* to the underlying storage, be that a memory block for
     // Shared Memory, or a GpuMemoryBuffer.
     virtual void* storage() = 0;
     // Amount of bytes requested when first created. Can be zero if it does not
     // need RAM, e.g. is allocated in GPU memory.
     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;
 
    private:
     // Indicates whether this Tracker is currently referenced by the producer.
     bool held_by_producer_;
     // Number of consumer processes which hold this Tracker.
     int consumer_hold_count_;
   };
@@ skipping 19 matching lines @@
   // The buffers, indexed by the first parameter, a buffer id.
   using TrackerMap = std::map<int, Tracker*>;
   TrackerMap trackers_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(VideoCaptureBufferPool);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_BUFFER_POOL_H_