Pepper: Expose visible_rect to PPB_VideoDecoder.GetPicture.

ppapi/proxy/video_decoder_resource_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <GLES2/gl2.h>
 
 #include "base/memory/shared_memory.h"
 #include "base/message_loop/message_loop.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/c/ppb_video_decoder.h"
@@ skipping 198 matching lines @@
     return result;
   }
 
   void SendDecodeReply(const ResourceMessageCallParams& params,
                        uint32_t shm_id) {
     SendReply(params, PP_OK, PpapiPluginMsg_VideoDecoder_DecodeReply(shm_id));
   }
 
   void SendPictureReady(const ResourceMessageCallParams& params,
                         uint32_t decode_count,
-                        uint32_t texture_id) {
-    SendReply(
-        params,
-        PP_OK,
-        PpapiPluginMsg_VideoDecoder_PictureReady(decode_count, texture_id));
+                        uint32_t texture_id,
+                        PP_Rect* visible_rect) {
+    SendReply(params, PP_OK, PpapiPluginMsg_VideoDecoder_PictureReady(
+                                 decode_count, texture_id, *visible_rect));
   }
 
   void SendFlushReply(const ResourceMessageCallParams& params) {
     SendReply(params, PP_OK, PpapiPluginMsg_VideoDecoder_FlushReply());
   }
 
   void SendResetReply(const ResourceMessageCallParams& params) {
     SendReply(params, PP_OK, PpapiPluginMsg_VideoDecoder_ResetReply());
   }
 
@@ skipping 190 matching lines @@
   ASSERT_FALSE(uncalled_cb.called());
   // Free up the first decode buffer.
   SendDecodeReply(params, 0U);
   // The decoder should run the pending callback.
   ASSERT_TRUE(decode_cb.called());
   ASSERT_EQ(PP_OK, decode_cb.result());
   decode_cb.Reset();
 
   // Now try to get a picture. No picture ready message has been received yet.
   PP_VideoPicture picture;
+  PP_Rect visible_rect;
   ASSERT_EQ(PP_OK_COMPLETIONPENDING,
             CallGetPicture(decoder.get(), &picture, &get_picture_cb));
   ASSERT_FALSE(get_picture_cb.called());
   // Calling GetPicture when another GetPicture is pending should fail.
   ASSERT_EQ(PP_ERROR_INPROGRESS,
             CallGetPicture(decoder.get(), &picture, &uncalled_cb));
   ASSERT_FALSE(uncalled_cb.called());
   // Send 'request textures' message to initialize textures.
   SendRequestTextures(params);
   // Send a picture ready message for Decode call 1. The GetPicture callback
   // should complete.
-  SendPictureReady(params, 1U, kTextureId1);
+  SendPictureReady(params, 1U, kTextureId1, &visible_rect);
   ASSERT_TRUE(get_picture_cb.called());
   ASSERT_EQ(PP_OK, get_picture_cb.result());
   ASSERT_EQ(kDecodeId, picture.decode_id);
   get_picture_cb.Reset();
 
   // Send a picture ready message for Decode call 2. Since there is no pending
   // GetPicture call, the picture should be queued.
-  SendPictureReady(params, 2U, kTextureId2);
+  SendPictureReady(params, 2U, kTextureId2, &visible_rect);
   // The next GetPicture should return synchronously.
   ASSERT_EQ(PP_OK, CallGetPicture(decoder.get(), &picture, &uncalled_cb));
   ASSERT_FALSE(uncalled_cb.called());
   ASSERT_EQ(kDecodeId, picture.decode_id);
 }
 #endif  // !defined(OS_WIN) || !defined(ARCH_CPU_64_BITS)
 
 // TODO(bbudge) Fix sync message testing on Windows 64 bit builds. The reply
 // message for GetShm isn't received, causing Decode to fail.
 // http://crbug.com/379260
 #if !defined(OS_WIN) || !defined(ARCH_CPU_64_BITS)
 TEST_F(VideoDecoderResourceTest, RecyclePicture) {
   LockingResourceReleaser decoder(CreateAndInitializeDecoder());
   ResourceMessageCallParams params;
   MockCompletionCallback decode_cb, get_picture_cb, uncalled_cb;
 
   // Get to a state where we have a picture to recycle.
   PpapiHostMsg_VideoDecoder_GetShm shm_msg(0U, kDecodeBufferSize);
   ASSERT_EQ(PP_OK, CallDecode(decoder.get(), &decode_cb, &shm_msg));
   uint32_t shm_id;
   uint32_t decode_size;
   int32_t decode_id;
   CheckDecodeMsg(&params, &shm_id, &decode_size, &decode_id);
   SendDecodeReply(params, 0U);
   // Send 'request textures' message to initialize textures.
   SendRequestTextures(params);
   // Call GetPicture and send 'picture ready' message to get a picture to
   // recycle.
   PP_VideoPicture picture;
+  PP_Rect visible_rect;
   ASSERT_EQ(PP_OK_COMPLETIONPENDING,
             CallGetPicture(decoder.get(), &picture, &get_picture_cb));
-  SendPictureReady(params, 0U, kTextureId1);
+  SendPictureReady(params, 0U, kTextureId1, &visible_rect);
   ASSERT_EQ(kTextureId1, picture.texture_id);
 
   CallRecyclePicture(decoder.get(), picture);
   uint32_t texture_id;
   ASSERT_TRUE(CheckRecyclePictureMsg(&params, &texture_id));
   ASSERT_EQ(kTextureId1, texture_id);
 
   ClearCallbacks(decoder.get());
 }
 #endif  // !defined(OS_WIN) || !defined(ARCH_CPU_64_BITS)
@@ skipping 77 matching lines @@
   ASSERT_FALSE(uncalled_cb.called());
   ASSERT_EQ(PP_ERROR_RESOURCE_FAILED, CallFlush(decoder.get(), &uncalled_cb));
   ASSERT_FALSE(uncalled_cb.called());
   ASSERT_EQ(PP_ERROR_RESOURCE_FAILED, CallReset(decoder.get(), &uncalled_cb));
   ASSERT_FALSE(uncalled_cb.called());
 }
 #endif  // !defined(OS_WIN) || !defined(ARCH_CPU_64_BITS)
 
 }  // namespace proxy
 }  // namespace ppapi