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

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

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_deferred_rendering_backing_strategy.cc
diff --git a/content/common/gpu/media/android_deferred_rendering_backing_strategy.cc b/content/common/gpu/media/android_deferred_rendering_backing_strategy.cc
index 78472bc239bf977ca90a00591afb4aa504cd515a..4075c9697b80dff0ef5a07169818e4b699067fa4 100644
--- a/content/common/gpu/media/android_deferred_rendering_backing_strategy.cc
+++ b/content/common/gpu/media/android_deferred_rendering_backing_strategy.cc
@@ -18,11 +18,6 @@
namespace content {
-// TODO(liberato): This is an entirely made-up number. It depends on how
-// many decoded buffers that the MediaCodec is willing to have outstanding
-// at any one time. Only one is guaranteed. crbug.com/531606.
-enum { kNumPictureBuffers = 3 };
-
AndroidDeferredRenderingBackingStrategy::
AndroidDeferredRenderingBackingStrategy()
: state_provider_(nullptr), media_codec_(nullptr) {}
@@ -47,10 +42,6 @@ void AndroidDeferredRenderingBackingStrategy::Cleanup(
}
}
-uint32 AndroidDeferredRenderingBackingStrategy::GetNumPictureBuffers() const {
- return kNumPictureBuffers;
-}
-
uint32 AndroidDeferredRenderingBackingStrategy::GetTextureTarget() const {
return GL_TEXTURE_EXTERNAL_OES;
}

Powered by Google App Engine
This is Rietveld 408576698