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());
+  if (dpi.x()) {

[Wez, 2012/07/19 23:15:29]

nit: In the host code you've used single-line if without {}; since these are the
only single line ifs in this method, you could do the same here.

[Jamie, 2012/07/20 00:42:14]

I only did that for consistency with the rest of that file. I prefer this style
given a choice. However, no-braces seems marginally more prevalent in this file,
so I'll change it.

+    data->SetInteger("x_dpi", dpi.x());
+  }
+  if (dpi.y()) {
+    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