Improving the decoder pipeline.

remoting/client/plugin/chromoting_instance.cc

 // Copyright (c) 2012 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/plugin/chromoting_instance.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 170 matching lines @@
   GetScriptableObject()->SetConnectionStatus(
       ChromotingScriptableObject::STATUS_CLOSED,
       ChromotingScriptableObject::ERROR_NONE);
 }
 
 void ChromotingInstance::DidChangeView(const pp::Rect& position,
                                        const pp::Rect& clip) {
   DCHECK(plugin_message_loop_->BelongsToCurrentThread());
 
   SkISize new_size = SkISize::Make(position.width(), position.height());
-  if (view_->SetViewSize(new_size)) {
+  SkIRect new_clip =
+    SkIRect::MakeXYWH(clip.x(), clip.y(), clip.width(), clip.height());
+
+  if (view_->SetView(new_size, new_clip)) {
     if (mouse_input_filter_.get()) {
       mouse_input_filter_->set_input_size(new_size);
     }
-    rectangle_decoder_->SetOutputSize(new_size);
+    // TODO(alexeypa): Update only the changed area if possible.
+    rectangle_decoder_->RefreshFullFrame();
   }
+}
 
-  rectangle_decoder_->UpdateClipRect(
-      SkIRect::MakeXYWH(clip.x(), clip.y(), clip.width(), clip.height()));
+void ChromotingInstance::RefreshFullFrame() {
+  rectangle_decoder_->RefreshFullFrame();
 }
 
 bool ChromotingInstance::HandleInputEvent(const pp::InputEvent& event) {
   DCHECK(plugin_message_loop_->BelongsToCurrentThread());
 
   // Never inject events if the end of the input pipeline doesn't exist.
   // If it does exist but the pipeline doesn't, construct a pipeline.
   // TODO(wez): This is really ugly.  We should create the pipeline when
   // the ConnectionToHost's InputStub exists.
   if (!host_connection_.get()) {
     return false;
   } else if (!input_handler_.get()) {
     protocol::InputStub* input_stub = host_connection_->input_stub();
     if (!input_stub)
       return false;
     mouse_input_filter_.reset(new MouseInputFilter(input_stub));
     mouse_input_filter_->set_input_size(view_->get_view_size());
     key_event_tracker_.reset(
         new protocol::KeyEventTracker(mouse_input_filter_.get()));
     input_handler_.reset(
         new PepperInputHandler(key_event_tracker_.get()));
   }
 
   // TODO(wez): When we have a good hook into Host dimensions changes, move
   // this there.
-  mouse_input_filter_->set_output_size(view_->get_host_size());
+  mouse_input_filter_->set_output_size(view_->get_screen_size());
 
   return input_handler_->HandleInputEvent(event);
 }
 
 ChromotingScriptableObject* ChromotingInstance::GetScriptableObject() {
   pp::VarPrivate object = GetInstanceObject();
   if (!object.is_undefined()) {
     pp::deprecated::ScriptableObject* so = object.AsScriptableObject();
     DCHECK(so != NULL);
     return static_cast<ChromotingScriptableObject*>(so);
@@ skipping 116 matching lines @@
   return client_->GetStats();
 }
 
 void ChromotingInstance::ReleaseAllKeys() {
   if (key_event_tracker_.get()) {
     key_event_tracker_->ReleaseAllKeys();
   }
 }
 
 }  // namespace remoting