Add workaround for PPB_VideoDecoder not setting visible_rect.

remoting/client/plugin/pepper_video_renderer_3d.h

 // 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.
 
 #ifndef REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_
 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_
 
 #include <deque>
 #include <string>
 
@@ skipping 80 matching lines @@
   protocol::PerformanceTracker* perf_tracker_ = nullptr;
 
   pp::Graphics3D graphics_;
   const PPB_OpenGLES2* gles2_if_;
   pp::VideoDecoder video_decoder_;
 
   webrtc::DesktopSize frame_size_;
   webrtc::DesktopVector frame_dpi_;
   scoped_ptr<webrtc::DesktopRegion> frame_shape_;
 
+  // Sizes for the frames in the pipe. Necessary to workaround
+  // crbug.com/542945 . Set only when decoder returns frames that don't have
+  // visible_rect set.
+  typedef std::map<uint32_t, webrtc::DesktopSize> FrameSizesMap;
+  scoped_ptr<FrameSizesMap> frame_sizes_;

[Wez, 2015/10/19 21:56:16]

Do you need something this complicated? Frame sizes don't change very often, so
surely you can set the size to the minimum of the texture dimensions and the
most-recent frame size from ProcessVideoPacket? You might then get artefacts on
resize, but only for a single frame.

[Sergey Ulanov, 2015/10/19 22:19:51]

Yes, good point. Done.

+
   webrtc::DesktopSize view_size_;
 
   bool initialization_finished_ = false;
   bool decode_pending_ = false;
   bool get_picture_pending_ = false;
   bool paint_pending_ = false;
 
   // Queue of packets that that have been received, but haven't been passed to
   // the decoder yet.
   std::deque<PendingPacket*> pending_packets_;
@@ skipping 25 matching lines @@
   bool debug_dirty_region_ = false;
 
   pp::CompletionCallbackFactory<PepperVideoRenderer3D> callback_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperVideoRenderer3D);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_