VaapiVideoAccelerator: make Vaapi accelerator work with ozone

content/common/gpu/media/vaapi_picture.h

+// Copyright 2014 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.
+//
+// This file contains an interface of output pictures for the Vaapi
+// video decoder. This is implemented by different window system
+// (X11/Ozone) and used by VaapiVideoDecodeAccelerator to produce
+// output pictures.
+
+#ifndef CONTENT_COMMON_GPU_MEDIA_VAAPI_PICTURE_H_
+#define CONTENT_COMMON_GPU_MEDIA_VAAPI_PICTURE_H_
+
+#include "base/callback.h"
+#include "base/memory/linked_ptr.h"
+#include "base/memory/ref_counted.h"
+#include "base/threading/non_thread_safe.h"
+#include "ui/gfx/size.h"
+
+namespace gfx {
+class GLContext;
+}  // namespace gfx
+
+namespace content {
+
+class VASurface;
+class VaapiWrapper;
+
+// Picture is native pixmap abstraction (X11/Ozone).
+class VaapiPicture : public base::NonThreadSafe {
+ public:
+  virtual ~VaapiPicture() {}
+
+  // Try to allocate the underlying resources for the picture.
+  virtual bool Initialize() = 0;
+
+  int32 picture_buffer_id() const { return picture_buffer_id_; }
+  uint32 texture_id() const { return texture_id_; }
+  const gfx::Size& size() const { return size_; }
+
+  // Downloads the |va_surface| into the picture, potentially scaling
+  // it if needed.
+  virtual bool DownloadFromSurface(
+      const scoped_refptr<VASurface>& va_surface) = 0;
+
+  // Create a VaapiPicture of |size| to be associated with
+  // |picture_buffer_id| and bound to |texture_id|. |gl_context| and
+  // |make_context_current| are provided for the GL operations.
+  static linked_ptr<VaapiPicture> CreatePicture(

[Pawel Osciak, 2014/12/09 01:19:25]

Can this return scoped_ptr?

[llandwerlin-old, 2014/12/09 11:19:48]

I thought the purpose of using the linked_ptr was to avoid all the Pass() calls
involved with scoped_ptr and the problems related to storage inside std::
containers.

I can give a try after the rebase.

[Pawel Osciak, 2014/12/10 01:15:40]

Yeah, although I was thinking to make this more generic, but nevermind, we can
keep this.

+      scoped_refptr<VaapiWrapper> vaapi_wrapper,
+      gfx::GLContext* gl_context,
+      const base::Callback<bool(void)> make_context_current,
+      int32 picture_buffer_id,
+      uint32 texture_id,
+      const gfx::Size& size);
+
+  // Get the texture target used to bind EGLImages (either
+  // GL_TEXTURE_2D on X11 or GL_TEXTURE_EXTERNAL_OES on DRM).
+  static uint32 GetGLTextureTarget();
+
+ protected:
+  VaapiPicture(int32 picture_buffer_id,
+               uint32 texture_id,
+               const gfx::Size& size)
+      : picture_buffer_id_(picture_buffer_id),
+        texture_id_(texture_id),
+        size_(size) {}
+
+ private:
+  int32 picture_buffer_id_;
+  uint32 texture_id_;
+  gfx::Size size_;
+
+  DISALLOW_COPY_AND_ASSIGN(VaapiPicture);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_COMMON_GPU_MEDIA_VAAPI_PICTURE_H_