Clean up video frame sizes, types in Video Decode API

media/video/picture.h

Index: media/video/picture.h
diff --git a/media/video/picture.h b/media/video/picture.h
index f9611bd7afc8e995fe64c13bbc5f489e73301370..1c24fc8483e8b21272831a71ad934c2ab3971e7c 100644
--- a/media/video/picture.h
+++ b/media/video/picture.h
@@ -93,8 +93,7 @@ class SysmemBuffer {
 // This is the media-namespace equivalent of PP_Picture_Dev.
 class Picture {
  public:
-  Picture(int32 picture_buffer_id, int32 bitstream_buffer_id,
-          gfx::Size visible_size, gfx::Size decoded_size);
+  Picture(int32 picture_buffer_id, int32 bitstream_buffer_id);
   Picture(const PP_Picture_Dev& picture);
 
   // Returns the id of the picture buffer where this picture is contained.
@@ -108,21 +107,9 @@ class Picture {
     return bitstream_buffer_id_;
   }
 
-  // Returns the visible size of the decoded picture in pixels.
-  gfx::Size visible_size() const {
-    return visible_size_;
-  }
-
-  // Returns the decoded size of the decoded picture in pixels.
-  gfx::Size decoded_size() const {
-    return decoded_size_;
-  }
-
  private:
   int32 picture_buffer_id_;
   int32 bitstream_buffer_id_;
-  gfx::Size visible_size_;
-  gfx::Size decoded_size_;

[Ville-Mikko Rautio, 2011/05/17 08:58:49]

After removing these how do we plan to handle situation where the picture size
changes due to in-band signalling in video bitstream? This means a case where
application does not see the change in picture parameters, only the decoder gets
to know about it through decoding the input video bitstream with new parameters.

This is valid use case in most modern video coding standards, including H.264
and VP8.

[vrk (LEFT CHROMIUM), 2011/05/23 18:20:29]

If the decoded picture size changes, the decoder will dismiss the currently
assigned picture buffers and will request new picture buffers of the proper
size.

The output size is still always determined by the client.

 };
 
 }  // namespace media