Refactoring of the client-side input pipeline and scaling dimension management.

remoting/client/rectangle_update_decoder.cc

 // Copyright (c) 2011 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 "remoting/client/rectangle_update_decoder.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "remoting/base/decoder.h"
@@ skipping 111 matching lines @@
   if (!decoder_->IsReadyForData()) {
     // TODO(ajwong): This whole thing should move into an invalid state.
     LOG(ERROR) << "Decoder is unable to process data. Dropping packet.";
     return;
   }
 
   if (decoder_->DecodePacket(packet) == Decoder::DECODE_DONE)
     SubmitToConsumer();
 }
 
-void RectangleUpdateDecoder::SetScaleRatios(double horizontal_ratio,
-                                            double vertical_ratio) {
+void RectangleUpdateDecoder::SetOutputDimensions(const SkISize& size) {
   if (message_loop_ != MessageLoop::current()) {
     message_loop_->PostTask(
-        FROM_HERE, base::Bind(&RectangleUpdateDecoder::SetScaleRatios,
-                              this, horizontal_ratio, vertical_ratio));
+        FROM_HERE, base::Bind(&RectangleUpdateDecoder::SetOutputDimensions,
+                              this, size));
     return;
   }
 
   // TODO(hclam): If the scale ratio has changed we should reallocate a
   // VideoFrame of different size. However if the scale ratio is always
   // smaller than 1.0 we can use the same video frame.
-  decoder_->SetScaleRatios(horizontal_ratio, vertical_ratio);
+
+  // TODO(wez): Because we're not using a scaling ratio, we do need to
+  // reallocate the video frame here, and re-initialize the decoder.
+
+  if (decoder_.get()) {
+    decoder_->SetOutputDimensions(size);
+    RefreshFullFrame();
+  }
 }
 
 void RectangleUpdateDecoder::UpdateClipRect(const SkIRect& new_clip_rect) {
   if (message_loop_ != MessageLoop::current()) {
     message_loop_->PostTask(
         FROM_HERE, base::Bind(&RectangleUpdateDecoder::UpdateClipRect,
                               this, new_clip_rect));
     return;
   }
 
@@ skipping 89 matching lines @@
     return;
   }
 
   delete rects;
 
   frame_is_consuming_ = false;
   DoRefresh();
 }
 
 }  // namespace remoting