MJPEG acceleration for video capture using VAAPI

content/common/gpu/media/vaapi_jpeg_decode_accelerator.h

Index: content/common/gpu/media/vaapi_jpeg_decode_accelerator.h
diff --git a/content/common/gpu/media/vaapi_jpeg_decode_accelerator.h b/content/common/gpu/media/vaapi_jpeg_decode_accelerator.h
new file mode 100644
index 0000000000000000000000000000000000000000..6f4a8f8fbef6389b55306c8659c63c1bc044d664
--- /dev/null
+++ b/content/common/gpu/media/vaapi_jpeg_decode_accelerator.h
@@ -0,0 +1,132 @@
+// Copyright 2015 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_COMMON_GPU_MEDIA_VAAPI_JPEG_DECODE_ACCELERATOR_H_
+#define CONTENT_COMMON_GPU_MEDIA_VAAPI_JPEG_DECODE_ACCELERATOR_H_
+
+#include <queue>
+
+#include "base/memory/linked_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "base/message_loop/message_loop.h"
+#include "base/synchronization/lock.h"
+#include "base/threading/non_thread_safe.h"
+#include "base/threading/thread.h"
+#include "content/common/content_export.h"
+#include "content/common/gpu/media/vaapi_jpeg_decoder.h"
+#include "content/common/gpu/media/vaapi_wrapper.h"
+#include "media/base/bitstream_buffer.h"
+#include "media/video/jpeg_decode_accelerator.h"
+#include "media/video/picture.h"
+
+namespace content {
+
+// Class to provide JPEG decode acceleration for Intel systems with hardware
+// support for it, and on which libva is available.
+// Decoding tasks are performed in a separate decoding thread.
+//
+// Threading/life-cycle: this object is created & destroyed on the GPU
+// ChildThread.  A few methods on it are called on the decoder thread which is
+// stopped during |this->Destroy()|, so any tasks posted to the decoder thread
+// can assume |*this| is still alive.  See |weak_this_| below for more details.
+class CONTENT_EXPORT VaapiJpegDecodeAccelerator
+    : public media::JpegDecodeAccelerator {
+ public:
+  VaapiJpegDecodeAccelerator(
+      const scoped_refptr<base::MessageLoopProxy>& io_message_loop);
+  ~VaapiJpegDecodeAccelerator() override;
+
+  // media::JpegDecodeAccelerator implementation.
+  bool Initialize(media::JpegDecodeAccelerator::Client* client) override;
+  void Decode(const media::BitstreamBuffer& bitstream_buffer,
+              const scoped_refptr<media::VideoFrame>& video_frame) override;
+  void Destroy() override;
+
+ private:
+  // Notifies the client that an error has occurred and decoding cannot
+  // continue.
+  void NotifyError(int32_t bitstream_buffer_id, Error error);
+
+  // Processes one decode request in the queue.
+  void DecodeTask();
+
+  // Helper for Destroy(), doing all the actual work except for deleting self.
+  void Cleanup();
+
+  // Puts contents of |va_surface| into given |video_frame|, releases the
+  // surface and passes the buffer id of the resulting picture to client for
+  // output.
+  bool OutputPicture(VASurfaceID va_surface_id,
+                     int32_t input_buffer_id,
+                     const scoped_refptr<media::VideoFrame>& video_frame);
+
+  // VAVDA state.
+  enum State {

[wuchengli, 2015/05/13 15:04:07]

remove

[kcwu, 2015/05/25 18:20:36]

Done.

+    // Initialize() not called yet or failed.
+    kUninitialized,
+    // Initialize() succeeded.
+    kInitialized,
+  };
+
+  // Protects |decode_requests_| and |initialized_|.
+  base::Lock lock_;
+  bool initialized_;
+
+  // An input buffer and the corresponding output video frame awaiting
+  // consumption, provided by the client.
+  struct DecodeRequest {
+    DecodeRequest(const media::BitstreamBuffer& bitstream_buffer,
+                  scoped_refptr<media::VideoFrame> video_frame);
+    ~DecodeRequest();
+
+    media::BitstreamBuffer bitstream_buffer;
+    scoped_refptr<media::VideoFrame> video_frame;
+  };
+
+  // Queue for incoming decode requests.
+  typedef std::queue<linked_ptr<DecodeRequest>> DecodeRequests;
+  DecodeRequests decode_requests_;
+
+  // ChildThread's message loop
+  base::MessageLoop* message_loop_;
+
+  // GPU IO message loop.
+  scoped_refptr<base::MessageLoopProxy> io_message_loop_;
+
+  // To expose client callbacks from JpegDecodeAccelerator.
+  // NOTE: all calls to these objects *MUST* be executed on message_loop_.

[wuchengli, 2015/05/13 15:04:07]

s/message_loop_/|message_loop_|/

[kcwu, 2015/05/25 18:20:35]

Done.

+  scoped_ptr<base::WeakPtrFactory<Client>> client_ptr_factory_;
+  base::WeakPtr<Client> client_;
+
+  // WeakPtr<> pointing to |this| for use in posting tasks from the decoder
+  // thread back to the ChildThread.  Because the decoder thread is a member of
+  // this class, any task running on the decoder thread is guaranteed that this
+  // object is still alive.  As a result, tasks posted from ChildThread to
+  // decoder thread should use base::Unretained(this), and tasks posted from the
+  // decoder thread to the ChildThread should use |weak_this_|.
+  base::WeakPtr<VaapiJpegDecodeAccelerator> weak_this_;
+
+  scoped_ptr<VaapiWrapper> vaapi_wrapper_;
+
+  // Comes after vaapi_wrapper_ to ensure its destructor is executed before
+  // vaapi_wrapper_ is destroyed.

[wuchengli, 2015/05/13 15:04:07]

s/vaapi_wrapper_/|vaapi_wrapper_|/

[kcwu, 2015/05/25 18:20:36]

Done.

+  scoped_ptr<VaapiJpegDecoder> decoder_;
+  base::Thread decoder_thread_;
+  // Use this to post tasks to |decoder_thread_| instead of
+  // |decoder_thread_.message_loop()| because the latter will be NULL once
+  // |decoder_thread_.Stop()| returns.
+  scoped_refptr<base::MessageLoopProxy> decoder_thread_proxy_;
+
+  VASurfaceID va_surface_;
+  gfx::Size last_coded_size_;

[wuchengli, 2015/05/13 15:04:07]

Add variable comment for these two variables.

[kcwu, 2015/05/25 18:20:35]

Done.

+
+  // The WeakPtrFactory for |weak_this_|.
+  base::WeakPtrFactory<VaapiJpegDecodeAccelerator> weak_this_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(VaapiJpegDecodeAccelerator);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_COMMON_GPU_MEDIA_VAAPI_JPEG_DECODE_ACCELERATOR_H_