Pass decoded picture size from VDA to client

content/renderer/pepper/video_decoder_shim.cc

Index: content/renderer/pepper/video_decoder_shim.cc
diff --git a/content/renderer/pepper/video_decoder_shim.cc b/content/renderer/pepper/video_decoder_shim.cc
index c52c285a10291e218bf3db516f915aa50fc8244e..9dc1cf691e0dd5b3119ad70d4a4b381f041cdfd7 100644
--- a/content/renderer/pepper/video_decoder_shim.cc
+++ b/content/renderer/pepper/video_decoder_shim.cc
@@ -527,7 +527,8 @@ void VideoDecoderShim::SendPictures() {
                       GL_UNSIGNED_BYTE,
                       &frame->argb_pixels.front());
 
-    host_->PictureReady(media::Picture(texture_id, frame->decode_id));
+    host_->PictureReady(media::Picture(texture_id, frame->decode_id,
+                                       texture_size_));

[Pawel Osciak, 2014/08/10 00:02:21]

We should be passing visible size here, not coded size. You should be able to
get visible size from VideoFrame that comes into OnOutputComplete.

[kcwu, 2014/08/12 04:48:06]

I don't understand what you mean. texture_size_ here comes from
PendingFrame->size in OnOutputComplete (line 498). There is only one size in
PendingFrame.

[Pawel Osciak, 2014/08/12 04:59:24]

OnOutputComplete gets a VF, which has both coded and visible size. We only use
coded size from it. We should keep using coded size for texture/buffer size, but
pass visible size to PictureReady.

[kcwu, 2014/08/12 07:31:11]

Ah, I was confused with VideoDecoderShim::DecoderImpl::OnOutputComplete and
VideoDecoderShim::OnOutputComplete.
Anyway, done.

     pending_frames_.pop();
   }