Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(85)

Unified Diff: content/common/gpu/media/android_deferred_rendering_backing_strategy.h

Issue 1892013002: ReleaseOutputBuffer to surface back and front buffers where possible. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/gpu/media/android_deferred_rendering_backing_strategy.h
diff --git a/content/common/gpu/media/android_deferred_rendering_backing_strategy.h b/content/common/gpu/media/android_deferred_rendering_backing_strategy.h
index 6809f4dddc853665eda0881408ae9828d4ee52e9..def6edb70c0bb71565c842770531bbed76ffdb05 100644
--- a/content/common/gpu/media/android_deferred_rendering_backing_strategy.h
+++ b/content/common/gpu/media/android_deferred_rendering_backing_strategy.h
@@ -6,6 +6,7 @@
#define CONTENT_COMMON_GPU_MEDIA_ANDROID_DEFERRED_RENDERING_BACKING_STRATEGY_H_
#include <stdint.h>
+#include <vector>
#include "base/macros.h"
#include "content/common/content_export.h"
@@ -40,16 +41,20 @@ class CONTENT_EXPORT AndroidDeferredRenderingBackingStrategy
// AndroidVideoDecodeAccelerator::BackingStrategy
gfx::ScopedJavaSurface Initialize(int surface_view_id) override;
- void Cleanup(bool have_context,
- const AndroidVideoDecodeAccelerator::OutputBufferMap&) override;
+ void Cleanup(
+ bool have_context,
+ const AndroidVideoDecodeAccelerator::OutputBufferMap& buffers) override;
scoped_refptr<gfx::SurfaceTexture> GetSurfaceTexture() const override;
uint32_t GetTextureTarget() const override;
gfx::Size GetPictureBufferSize() const override;
- void UseCodecBufferForPictureBuffer(int32_t codec_buffer_index,
- const media::PictureBuffer&) override;
+ void UseCodecBufferForPictureBuffer(
+ int32_t codec_buffer_index,
+ const media::PictureBuffer& picture_buffer) override;
void AssignOnePictureBuffer(const media::PictureBuffer&, bool) override;
- void ReuseOnePictureBuffer(const media::PictureBuffer&) override;
- void CodecChanged(media::VideoCodecBridge*) override;
+ void ReuseOnePictureBuffer(
+ const media::PictureBuffer& picture_buffer) override;
+ void MaybeRenderEarly() override;
+ void CodecChanged(media::VideoCodecBridge* codec) override;
void ReleaseCodecBuffers(
const AndroidVideoDecodeAccelerator::OutputBufferMap& buffers) override;
void OnFrameAvailable() override;
@@ -94,6 +99,10 @@ class CONTENT_EXPORT AndroidDeferredRenderingBackingStrategy
media::VideoCodecBridge* media_codec_;
+ // Picture buffer IDs that are out for display. Stored in order of frames as
+ // they are returned from the decoder.
+ std::vector<int32_t> pictures_out_for_display_;
+
DISALLOW_COPY_AND_ASSIGN(AndroidDeferredRenderingBackingStrategy);
};

Powered by Google App Engine
This is Rietveld 408576698