Improving the decoder pipeline.

remoting/webapp/client_session.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 handling creation and teardown of a remoting client session.
  *
  * This abstracts a <embed> element and controls the plugin which does the
  * actual remoting work.  There should be no UI code inside this class.  It
@@ skipping 417 matching lines @@
  *
  * @return {void} Nothing.
  */
 remoting.ClientSession.prototype.updateDimensions = function() {
   if (this.plugin.desktopWidth == 0 ||
       this.plugin.desktopHeight == 0)
     return;
 
   var windowWidth = window.innerWidth;
   var windowHeight = window.innerHeight;
-  var scale = 1.0;
+
+  // Estimate browser scale factor in order to be able to translate logical
+  // pixels on the page into physical pixels on the screen. Scale factor can
+  // be roughly estimated as a ratio between outerWidht and innerWidth of
+  // the document view. The result is not precise because outerWidth can
+  // include pixels allocated for toolbars, window borders and such.
+  var viewInnerWidth = document.defaultView.innerWidth;
+  var viewOuterWidth = document.defaultView.outerWidth;
+  var scale = Math.round(100.0 * viewInnerWidth / viewOuterWidth) / 100.0;

[Wez, 2012/02/07 01:56:31]

This looks like a change from a different CL, so I'm skipping it.

 
   if (this.getScaleToFit()) {
-    var scaleFitHeight = 1.0 * windowHeight / this.plugin.desktopHeight;
-    var scaleFitWidth = 1.0 * windowWidth / this.plugin.desktopWidth;
-    scale = Math.min(1.0, scaleFitHeight, scaleFitWidth);
+    var scaledHeight = 1.0 * windowHeight / (scale * this.plugin.desktopHeight);
+    var scaledWidth = 1.0 * windowWidth / (scale * this.plugin.desktopWidth);
+    scale = Math.min(1.0, scaledHeight, scaledWidth, scale);
   }
 
   // Resize the plugin if necessary.
   this.plugin.width = this.plugin.desktopWidth * scale;
   this.plugin.height = this.plugin.desktopHeight * scale;
 
   // Position the container.
   // TODO(wez): We should take into account scrollbars when positioning.
   var parentNode = this.plugin.parentNode;
   if (this.plugin.width < windowWidth)
@@ skipping 37 matching lines @@
 };
 
 /**
  * Logs statistics.
  *
  * @param {Object.<string, number>} stats
  */
 remoting.ClientSession.prototype.logStatistics = function(stats) {
   this.logToServer.logStatistics(stats, this.mode);
 };