Propagate DPI information to web-app.

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
  * client plugins versions when it is necessary.
  */
 
 'use strict';
 
 /** @suppress {duplicate} */
 var remoting = remoting || {};
 
 /**
  * @param {remoting.ViewerPlugin} plugin The plugin embed element.
  * @constructor
  * @implements {remoting.ClientPlugin}
  */
 remoting.ClientPluginAsync = function(plugin) {
   this.plugin = plugin;
 
   this.desktopWidth = 0;
   this.desktopHeight = 0;
+  this.desktopDpiX = 0;
+  this.desktopDpiY = 0;
 
   /** @param {string} iq The Iq stanza received from the host. */
   this.onOutgoingIqHandler = function (iq) {};
   /** @param {string} message Log message. */
   this.onDebugMessageHandler = function (message) {};
   /**
    * @param {number} state The connection state.
    * @param {number} error The error code, if any.
    */
   this.onConnectionStatusUpdateHandler = function(state, error) {};
@@ skipping 116 matching lines @@
 
     this.onConnectionStatusUpdateHandler(state, error);
   } else if (message.method == 'onDesktopSize') {
     if (typeof message.data['width'] != 'number' ||
         typeof message.data['height'] != 'number') {
       console.error('Received invalid onDesktopSize message: ' + messageStr);
       return;
     }
     this.desktopWidth = /** @type {number} */ message.data['width'];
     this.desktopHeight = /** @type {number} */ message.data['height'];
+    this.desktopDpiX = (typeof message.data['x_dpi'] == 'number') ?
+        /** @type {number} */ (message.data['x_dpi']) : 0;

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

nit: Treat missing DPI as 96 DPI for consistency with the behaviour when
connecting to a pre-DPI host.

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

That was left-over debugging code. Fixed here and in the ctor.

+    this.desktopDpiY = (typeof message.data['y_dpi'] == 'number') ?
+        /** @type {number} */ (message.data['y_dpi']) : 0;
     this.onDesktopSizeUpdateHandler();
   } else if (message.method == 'onPerfStats') {
     if (typeof message.data['videoBandwidth'] != 'number' ||
         typeof message.data['videoFrameRate'] != 'number' ||
         typeof message.data['captureLatency'] != 'number' ||
         typeof message.data['encodeLatency'] != 'number' ||
         typeof message.data['decodeLatency'] != 'number' ||
         typeof message.data['renderLatency'] != 'number' ||
         typeof message.data['roundtripLatency'] != 'number') {
       console.error('Received incorrect onPerfStats message: ' + messageStr);
@@ skipping 221 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';
   }
 };