Update web-app client to use plugin notifyDeviceResolution API.

remoting/webapp/client_plugin_async.js

 // 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.
 
 /**
  * @fileoverview
  * Class that wraps low-level details of interacting with the client plugin.
  *
  * This abstracts a <embed> element and controls the plugin which does
  * the actual remoting work. It also handles differences between
@@ skipping 367 matching lines @@
 remoting.ClientPluginAsync.prototype.sendClipboardItem =
     function(mimeType, item) {
   if (!this.hasFeature(remoting.ClientPlugin.Feature.SEND_CLIPBOARD_ITEM))
     return;
   this.plugin.postMessage(JSON.stringify(
       { method: 'sendClipboardItem',
         data: { mimeType: mimeType, item: item }}));
 };
 
 /**
- * Notifies the host that the client has the specified dimensions.
+ * Notifies the host that the client has the specified size and pixel density.
  *
- * @param {number} width The available client width.
- * @param {number} height The available client height.
+ * @param {number} width The available client width in DIPs.
+ * @param {number} height The available client height in DIPs.
+ * @param {number} device_scale The number of device pixels per DIP.

[Jamie, 2013/02/11 20:32:02]

Does this depend on the version of the plugin? For older versions, we don't send
the dpi, and I'm not sure if the width and height are expressed in DIPs in that
case. Please update the comment to explain the former point (and the latter, if
relevant).

[Wez, 2013/02/11 23:05:33]

See below - if the plugin doesn't support notifyClientResolution then we use
notifyClientDimensions, which just sends the width and height in DIPs.

Under the hood notifyClientResolution adds the "legacy" DIPs-based width &
height.

  */
-remoting.ClientPluginAsync.prototype.notifyClientDimensions =
-    function(width, height) {
-  if (!this.hasFeature(remoting.ClientPlugin.Feature.NOTIFY_CLIENT_DIMENSIONS))
-    return;
-  this.plugin.postMessage(JSON.stringify(
-      { method: 'notifyClientDimensions',
-        data: { width: width, height: height }}));
+remoting.ClientPluginAsync.prototype.notifyClientResolution =
+    function(width, height, device_scale) {
+  if (this.hasFeature(remoting.ClientPlugin.Feature.NOTIFY_CLIENT_RESOLUTION)) {
+    var dpi = device_scale * 96;
+    this.plugin.postMessage(JSON.stringify(
+        { method: 'notifyClientResolution',
+          data: { width: width * device_scale,
+                  height: height * device_scale,
+                  x_dpi: dpi, y_dpi: dpi }}));
+  } else if (this.hasFeature(
+                 remoting.ClientPlugin.Feature.NOTIFY_CLIENT_DIMENSIONS)) {
+    this.plugin.postMessage(JSON.stringify(
+        { method: 'notifyClientDimensions',
+          data: { width: width, height: height }}));
+  }
 };
 
 /**
  * Requests that the host pause or resume sending video updates.
  *
  * @param {boolean} pause True to suspend video updates, false otherwise.
  */
 remoting.ClientPluginAsync.prototype.pauseVideo =
     function(pause) {
   if (!this.hasFeature(remoting.ClientPlugin.Feature.PAUSE_VIDEO))
@@ skipping 28 matching lines @@
     this.plugin.width = width;
     this.plugin.height = height;
     // Center the plugin just underneath the "Connnecting..." dialog.
     var parentNode = this.plugin.parentNode;
     var dialog = document.getElementById('client-dialog');
     var dialogRect = dialog.getBoundingClientRect();
     parentNode.style.top = (dialogRect.bottom + 16) + 'px';
     parentNode.style.left = (window.innerWidth - width) / 2 + 'px';
   }
 };