Make Chromoting client plugin always render at device DPI.

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 304 matching lines @@
   } else if (method == "pauseVideo") {
     bool pause = false;
     if (!data->GetBoolean("pause", &pause)) {
       LOG(ERROR) << "Invalid pauseVideo.";
       return;
     }
     PauseVideo(pause);
   }
 }
 
-void ChromotingInstance::DidChangeView(const pp::Rect& position,
-                                       const pp::Rect& clip) {
+void ChromotingInstance::DidChangeView(const pp::View& view) {
   DCHECK(plugin_message_loop_->BelongsToCurrentThread());
 
-  SkISize new_size = SkISize::Make(position.width(), position.height());
-  SkIRect new_clip =
-    SkIRect::MakeXYWH(clip.x(), clip.y(), clip.width(), clip.height());
-
-  view_->SetView(new_size, new_clip);
+  view_->SetView(view);
 
   if (mouse_input_filter_.get()) {
-    mouse_input_filter_->set_input_size(view_->get_view_size());
+    mouse_input_filter_->set_input_size(view_->get_view_size_dips());

[Sergey Ulanov, 2012/07/17 19:36:06]

Do we always send mouse position in DIPs?

[Wez, 2012/07/17 20:11:43]

We will always receive them from Pepper in DIPs, yes, although in future they
will be floats with sub-DIP resolution.

   }
 }
 
 bool ChromotingInstance::HandleInputEvent(const pp::InputEvent& event) {
   DCHECK(plugin_message_loop_->BelongsToCurrentThread());
 
   if (!IsConnected())
     return false;
 
   // TODO(wez): When we have a good hook into Host dimensions changes, move
@@ skipping 33 matching lines @@
 
   host_connection_.reset(new protocol::ConnectionToHost(true));
   client_.reset(new ChromotingClient(config, context_.main_task_runner(),
                                      host_connection_.get(), view_.get(),
                                      rectangle_decoder_.get(),
                                      audio_player_.get()));
 
   // Construct the input pipeline
   mouse_input_filter_.reset(
       new protocol::MouseInputFilter(host_connection_->input_stub()));
-  mouse_input_filter_->set_input_size(view_->get_view_size());
+  mouse_input_filter_->set_input_size(view_->get_view_size_dips());
   input_tracker_.reset(
       new protocol::InputEventTracker(mouse_input_filter_.get()));
 
 #if defined(OS_MACOSX)
   // On Mac we need an extra filter to inject missing keyup events.
   // See remoting/client/plugin/mac_key_event_processor.h for more details.
   mac_key_event_processor_.reset(
       new MacKeyEventProcessor(input_tracker_.get()));
   key_mapper_.set_input_stub(mac_key_event_processor_.get());
 #else
@@ skipping 309 matching lines @@
   PostChromotingMessage("logDebugMessage", data.Pass());
   g_logging_to_plugin = false;
 }
 
 bool ChromotingInstance::IsConnected() {
   return host_connection_.get() &&
     (host_connection_->state() == protocol::ConnectionToHost::CONNECTED);
 }
 
 }  // namespace remoting