Allow shaped-desktop hosts to send shape only when it changes.

remoting/client/software_video_renderer.cc

Index: remoting/client/software_video_renderer.cc
diff --git a/remoting/client/software_video_renderer.cc b/remoting/client/software_video_renderer.cc
index 51782e05f476f4942a30408a7e52d0214cc0a80f..f9df4dfff58dd06896c9b7b74b4f2eb58936496c 100644
--- a/remoting/client/software_video_renderer.cc
+++ b/remoting/client/software_video_renderer.cc
@@ -241,8 +241,12 @@ void SoftwareVideoRenderer::Core::DoPaint() {
   // Notify the consumer that painting is done.
   if (!output_region.is_empty()) {
     buffers_.pop_front();
-    consumer_->ApplyBuffer(view_size_, clip_area_, buffer, output_region,
-                           *decoder_->GetImageShape());
+    consumer_->ApplyBuffer(
+        view_size_, clip_area_, buffer, output_region,
+        decoder_->GetImageShape()

[Sergey Ulanov, 2015/07/10 21:51:43]

Maybe fix FrameConsumer interface to take pointer for frame shape?

[Wez, 2015/07/13 19:56:24]

Done.

As part of doing that I've also simplified FrameConsumerProxy and allowed the
PepperVideoRenderer EventHandler API to handle NULL shapes as well, to reduce
complexity.

+            ? *decoder_->GetImageShape()
+            : webrtc::DesktopRegion(
+                  webrtc::DesktopRect::MakeSize(source_size_)));
   }
 }