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

Unified Diff: ppapi/cpp/dev/video_decoder_dev.h

Issue 7085030: Implementation for Pepper C++ Video Decoder API (wrapper on top of C API). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Changed Assign*Buffers function parameters to std::vector type. Created 9 years, 7 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
« no previous file with comments | « no previous file | ppapi/cpp/dev/video_decoder_dev.cc » ('j') | ppapi/cpp/dev/video_decoder_dev.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/cpp/dev/video_decoder_dev.h
diff --git a/ppapi/cpp/dev/video_decoder_dev.h b/ppapi/cpp/dev/video_decoder_dev.h
index ba29abadcd285efebd69e65036d96fc169a21696..90823e501c80f031095f293055fe018551685885 100644
--- a/ppapi/cpp/dev/video_decoder_dev.h
+++ b/ppapi/cpp/dev/video_decoder_dev.h
@@ -70,10 +70,8 @@ class VideoDecoder : public Resource {
// Provides the decoder with picture buffers for video decoding.
// AssignGLESBuffers provides texture-backed buffers, whereas
// AssignSysmemBuffers provides system memory-backed buffers.
- void AssignGLESBuffers(uint32_t no_of_buffers,
- const PP_GLESBuffer_Dev& buffers);
- void AssignSysmemBuffers(uint32_t no_of_buffers,
- const PP_SysmemBuffer_Dev& buffers);
+ void AssignGLESBuffers(const std::vector<PP_GLESBuffer_Dev> buffers);
brettw 2011/05/31 12:46:44 The arguments to these two functions should be ref
Ville-Mikko Rautio 2011/05/31 13:33:14 Done.
+ void AssignSysmemBuffers(const std::vector<PP_SysmemBuffer_Dev> buffers);
// Decodes given bitstream buffer. Once decoder is done with processing
// |bitstream_buffer| is will call |callback| with provided user data.
« no previous file with comments | « no previous file | ppapi/cpp/dev/video_decoder_dev.cc » ('j') | ppapi/cpp/dev/video_decoder_dev.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698