Clean up video frame sizes, types in Video Decode API

webkit/plugins/ppapi/ppb_video_decoder_impl.cc

Index: webkit/plugins/ppapi/ppb_video_decoder_impl.cc
diff --git a/webkit/plugins/ppapi/ppb_video_decoder_impl.cc b/webkit/plugins/ppapi/ppb_video_decoder_impl.cc
index 0f887b8a408b7e320e39a0806472cf16b9f7dc4a..2491e4b76dafe5bc92feec4d4668b9f1d28c1a66 100644
--- a/webkit/plugins/ppapi/ppb_video_decoder_impl.cc
+++ b/webkit/plugins/ppapi/ppb_video_decoder_impl.cc
@@ -179,19 +179,13 @@ void PPB_VideoDecoder_Impl::Destroy() {
 }
 
 void PPB_VideoDecoder_Impl::ProvidePictureBuffers(
-    uint32 requested_num_of_buffers,
-    const gfx::Size& dimensions,
-    media::VideoDecodeAccelerator::MemoryType type) {
+    uint32 requested_num_of_buffers, const gfx::Size& dimensions) {
   if (!ppp_videodecoder_)
     return;
 
-  // TODO(vrk): Compiler assert or use switch statement instead of making
-  // a blind cast.

[Ami GONE FROM CHROMIUM, 2011/07/18 18:33:00]

Does this TODO not apply elsewhere?

[vrk (LEFT CHROMIUM), 2011/07/19 01:26:08]

Looks like there are TODOs written in all the other places where we do something
like this in this file.

-  PP_PictureBufferType_Dev out_type =
-      static_cast<PP_PictureBufferType_Dev>(type);
   PP_Size out_dim = PP_MakeSize(dimensions.width(), dimensions.height());
   ppp_videodecoder_->ProvidePictureBuffers(
-      instance()->pp_instance(), requested_num_of_buffers, out_dim, out_type);
+      instance()->pp_instance(), requested_num_of_buffers, out_dim);
 }
 
 void PPB_VideoDecoder_Impl::PictureReady(const media::Picture& picture) {
@@ -201,10 +195,6 @@ void PPB_VideoDecoder_Impl::PictureReady(const media::Picture& picture) {
   PP_Picture_Dev output;
   output.picture_buffer_id = picture.picture_buffer_id();
   output.bitstream_buffer_id = picture.bitstream_buffer_id();
-  output.visible_size = PP_MakeSize(picture.visible_size().width(),
-                                    picture.visible_size().height());
-  output.decoded_size = PP_MakeSize(picture.decoded_size().width(),
-                                    picture.decoded_size().height());
   ppp_videodecoder_->PictureReady(instance()->pp_instance(), output);
 }