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

Unified Diff: media/video/picture.h

Issue 426873004: Pass decoded picture size from VDA to client (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix build on mac Created 6 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/picture.h
diff --git a/media/video/picture.h b/media/video/picture.h
index d5be2276f2be78ad037bfd045798ebd7f629defb..6d1271ce5ab61dd79ae1cdff5d9f927a3509835b 100644
--- a/media/video/picture.h
+++ b/media/video/picture.h
@@ -54,7 +54,9 @@ class MEDIA_EXPORT PictureBuffer {
// This is the media-namespace equivalent of PP_Picture_Dev.
class MEDIA_EXPORT Picture {
public:
- Picture(int32 picture_buffer_id, int32 bitstream_buffer_id);
+ Picture(int32 picture_buffer_id,
+ int32 bitstream_buffer_id,
+ const gfx::Size& size);
// Returns the id of the picture buffer where this picture is contained.
int32 picture_buffer_id() const {
@@ -70,9 +72,15 @@ class MEDIA_EXPORT Picture {
bitstream_buffer_id_ = bitstream_buffer_id;
}
+ // Returns the size of the picture. This size may be smaller than the
+ // size of the PictureBuffer, as it is the visible size of the
+ // Picture contained in the PictureBuffer.
Ken Russell (switch to Gerrit) 2014/08/15 22:59:26 It's probably worth documenting how the picture is
kcwu 2014/08/18 07:02:04 Done.
+ gfx::Size size() const { return size_; }
+
private:
int32 picture_buffer_id_;
int32 bitstream_buffer_id_;
+ gfx::Size size_;
};
} // namespace media

Powered by Google App Engine
This is Rietveld 408576698