Propagate DPI information to web-app.

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 334 matching lines @@
 
   // TODO(wez): When we have a good hook into Host dimensions changes, move
   // this there.
   // If |input_handler_| is valid, then |mouse_input_filter_| must also be
   // since they are constructed together as part of the input pipeline
   mouse_input_filter_->set_output_size(view_->get_screen_size());
 
   return input_handler_->HandleInputEvent(event);
 }
 
-void ChromotingInstance::SetDesktopSize(int width, int height) {
+void ChromotingInstance::SetDesktopSize(const SkISize& size,
+                                        const SkIPoint& dpi) {
   scoped_ptr<base::DictionaryValue> data(new base::DictionaryValue());
-  data->SetInteger("width", width);
-  data->SetInteger("height", height);
+  data->SetInteger("width", size.width());
+  data->SetInteger("height", size.height());
+  data->SetInteger("x_dpi", dpi.x());

[Wez, 2012/07/19 01:00:42]

nit: I was going to suggest only passing x_dpi and y_xpi if they are non-zero,
i.e. if the host sent them, but I notice that you're having the plugin fake them
if they're not already present; is that worth it? There's not really any
difference between a client that doesn't understand the DPI fields (and so won't
include them here) and one that does, but is talking to an older host.

[Jamie, 2012/07/19 23:00:06]

Done.

+  data->SetInteger("y_dpi", dpi.y());
   PostChromotingMessage("onDesktopSize", data.Pass());
 }
 
 void ChromotingInstance::SetConnectionState(
     ConnectionState state,
     ConnectionError error) {
   scoped_ptr<base::DictionaryValue> data(new base::DictionaryValue());
   data->SetString("state", ConnectionStateToString(state));
   data->SetString("error", ConnectionErrorToString(error));
   PostChromotingMessage("onConnectionStatus", data.Pass());
@@ skipping 341 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