Chromium Code Reviews| Index: content/common/gpu/media/vaapi_wrapper.h |
| diff --git a/content/common/gpu/media/vaapi_wrapper.h b/content/common/gpu/media/vaapi_wrapper.h |
| index f600cdfa1d17198d00f4fd67b4b59767ea60795e..2018ca7753cfe4b6586b45ac93ee820d60e528a3 100644 |
| --- a/content/common/gpu/media/vaapi_wrapper.h |
| +++ b/content/common/gpu/media/vaapi_wrapper.h |
| @@ -14,13 +14,12 @@ |
| #include <vector> |
| #include "base/callback.h" |
| -#include "base/memory/ref_counted.h" |
| #include "base/synchronization/lock.h" |
| #include "content/common/content_export.h" |
| #include "content/common/gpu/media/va_surface.h" |
| #include "media/base/video_decoder_config.h" |
| #include "media/base/video_frame.h" |
| -#include "third_party/libva/va/va_x11.h" |
| +#include "third_party/libva/va/va.h" |
| #include "ui/gfx/size.h" |
| namespace content { |
| @@ -47,7 +46,6 @@ class CONTENT_EXPORT VaapiWrapper { |
| static scoped_ptr<VaapiWrapper> Create( |
| CodecMode mode, |
| media::VideoCodecProfile profile, |
| - Display* x_display, |
| const base::Closure& report_error_to_uma_cb); |
| ~VaapiWrapper(); |
| @@ -59,7 +57,7 @@ class CONTENT_EXPORT VaapiWrapper { |
| // again to free the allocated surfaces first, but is not required to do so |
| // at destruction time, as this will be done automatically from |
| // the destructor. |
| - bool CreateSurfaces(gfx::Size size, |
| + bool CreateSurfaces(const gfx::Size& size, |
| size_t num_surfaces, |
| std::vector<VASurfaceID>* va_surfaces); |
| @@ -92,12 +90,6 @@ class CONTENT_EXPORT VaapiWrapper { |
| // buffers. Return false if Execute() fails. |
| bool ExecuteAndDestroyPendingBuffers(VASurfaceID va_surface_id); |
| - // Put data from |va_surface_id| into |x_pixmap| of size |size|, |
| - // converting/scaling to it. |
| - bool PutSurfaceIntoPixmap(VASurfaceID va_surface_id, |
| - Pixmap x_pixmap, |
| - gfx::Size dest_size); |
| - |
| // Returns true if the VAAPI version is less than the specified version. |
| bool VAAPIVersionLessThan(int major, int minor); |
| @@ -134,12 +126,16 @@ class CONTENT_EXPORT VaapiWrapper { |
| // Destroy all previously-allocated (and not yet destroyed) coded buffers. |
| void DestroyCodedBuffers(); |
| + // Returns the initialized VADisplay |
| + VADisplay GetDisplay(); |
| + |
| private: |
| + |
|
Pawel Osciak
2014/09/24 11:27:13
Please don't add newlines.
|
| VaapiWrapper(); |
| - bool Initialize(CodecMode mode, |
| + bool Initialize(VADisplay display, |
| + CodecMode mode, |
| media::VideoCodecProfile profile, |
| - Display* x_display, |
| const base::Closure& report_error__to_uma_cb); |
| void Deinitialize(); |
| @@ -167,7 +163,7 @@ class CONTENT_EXPORT VaapiWrapper { |
| int major_version_, minor_version_; |
| // VA handles. |
| - // Both valid after successful Initialize() and until Deinitialize(). |
| + // All valid after successful Initialize() and until Deinitialize(). |
| VADisplay va_display_; |
| VAConfigID va_config_id_; |
| // Created for the current set of va_surface_ids_ in CreateSurfaces() and |