Ensure that dimensions and DPI are integers.

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 506 matching lines @@
 /**
  * Notifies the host that the client has the specified size and pixel density.
  *
  * @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.
  */
 remoting.ClientPluginAsync.prototype.notifyClientResolution =
     function(width, height, device_scale) {
   if (this.hasFeature(remoting.ClientPlugin.Feature.NOTIFY_CLIENT_RESOLUTION)) {
-    var dpi = device_scale * 96;
+    var dpi = Math.floor(device_scale * 96);
     this.plugin.postMessage(JSON.stringify(
         { method: 'notifyClientResolution',
-          data: { width: width * device_scale,
-                  height: height * device_scale,
+          data: { width: Math.floor(width * device_scale),
+                  height: Math.floor(height * device_scale),

[Lambros, 2013/11/08 22:26:33]

Math.round might be slightly better?

[Jamie, 2013/11/08 22:29:15]

I think we want floor. A 1px border is better than down-scaling or scrollbars.

                   x_dpi: dpi, y_dpi: dpi }}));
   }
 };
 
 /**
  * Requests that the host pause or resume sending video updates.
  *
  * @param {boolean} pause True to suspend video updates, false otherwise.
  */
 remoting.ClientPluginAsync.prototype.pauseVideo =
@@ skipping 103 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';
   }
 };