Propagate DPI information to web-app.

remoting/client/plugin/chromoting_instance.cc

Index: remoting/client/plugin/chromoting_instance.cc
diff --git a/remoting/client/plugin/chromoting_instance.cc b/remoting/client/plugin/chromoting_instance.cc
index 77cd55fdc4e10a4b4d8513f9701ef43f13215535..011fc54bf5812cddcb9316f5f78e8ea0dd1045e0 100644
--- a/remoting/client/plugin/chromoting_instance.cc
+++ b/remoting/client/plugin/chromoting_instance.cc
@@ -352,10 +352,17 @@ bool ChromotingInstance::HandleInputEvent(const pp::InputEvent& event) {
   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());
 }