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

Unified Diff: media/video/video_decode_accelerator.h

Issue 1207043002: Introduce a client minimum picture pool size (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Moved constant to shared header, validate min_picture_size now in resource proxy as well as host co… Created 5 years, 4 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: media/video/video_decode_accelerator.h
diff --git a/media/video/video_decode_accelerator.h b/media/video/video_decode_accelerator.h
index 53e18a786d569a4ce6565cd43eb795ab545cf186..2b71d65ab075926a56d690856d7de1b95a595406 100644
--- a/media/video/video_decode_accelerator.h
+++ b/media/video/video_decode_accelerator.h
@@ -61,6 +61,8 @@ class MEDIA_EXPORT VideoDecodeAccelerator {
class MEDIA_EXPORT Client {
public:
// Callback to tell client how many and what size of buffers to provide.
+ // Note that the actual count provided through AssignPictureBuffers() can be
+ // larger than the value requested.
virtual void ProvidePictureBuffers(uint32 requested_num_of_buffers,
const gfx::Size& dimensions,
uint32 texture_target) = 0;
@@ -115,7 +117,9 @@ class MEDIA_EXPORT VideoDecodeAccelerator {
// callback has been initiated for a given buffer.
//
// Parameters:
- // |buffers| contains the allocated picture buffers for the output.
+ // |buffers| contains the allocated picture buffers for the output. Note
+ // that the count of buffers may be larger than the count requested through
+ // the call to Client::ProvidePictureBuffers().
virtual void AssignPictureBuffers(
const std::vector<PictureBuffer>& buffers) = 0;
« no previous file with comments | « content/renderer/pepper/video_decoder_shim.cc ('k') | native_client_sdk/src/examples/api/video_decode/video_decode.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698