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

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

Issue 1490333005: Don't require VDAs to return all PictureBuffers at once. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cl feedback. Created 5 years 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_video_decode_accelerator.h
diff --git a/content/common/gpu/media/android_video_decode_accelerator.h b/content/common/gpu/media/android_video_decode_accelerator.h
index 945a4d0ac20f4c784b7e1d29c75e862212526833..6221ad933a4cc2477a9b75d9a801e7a06161f021 100644
--- a/content/common/gpu/media/android_video_decode_accelerator.h
+++ b/content/common/gpu/media/android_video_decode_accelerator.h
@@ -51,9 +51,6 @@ class CONTENT_EXPORT AndroidVideoDecodeAccelerator
// the last call that the BackingStrategy receives.
virtual void Cleanup(const OutputBufferMap& buffer_map) = 0;
- // Return the number of picture buffers that we can support.
- virtual uint32 GetNumPictureBuffers() const = 0;
-
// Return the GL texture target that the PictureBuffer textures use.
virtual uint32 GetTextureTarget() const = 0;
@@ -89,8 +86,7 @@ class CONTENT_EXPORT AndroidVideoDecodeAccelerator
AndroidVideoDecodeAccelerator(
const base::WeakPtr<gpu::gles2::GLES2Decoder> decoder,
- const base::Callback<bool(void)>& make_context_current,
- scoped_ptr<BackingStrategy> strategy);
+ const base::Callback<bool(void)>& make_context_current);
~AndroidVideoDecodeAccelerator() override;
@@ -113,8 +109,7 @@ class CONTENT_EXPORT AndroidVideoDecodeAccelerator
void PostError(const ::tracked_objects::Location& from_here,
media::VideoDecodeAccelerator::Error error) override;
- static media::VideoDecodeAccelerator::SupportedProfiles
- GetSupportedProfiles();
+ static media::VideoDecodeAccelerator::Capabilities GetCapabilities();
private:
enum State {

Powered by Google App Engine
This is Rietveld 408576698