Fix scaling calculation.

remoting/webapp/me2mom/client_session.js

 // Copyright (c) 2011 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
  * Session class that handles creation and teardown of a remoting 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 365 matching lines @@
     }
 
     // Make sure both width and height are multiples of two.
     var width = window.innerWidth;
     var height = window.innerHeight;
     if (width % 2 == 1)
       --width;
     if (height % 2 == 1)
       --height;
 
-    var scale = 1.0;
-    if (width < height)
-      scale = 1.0 * height / this.plugin.desktopHeight;
-    else
-      scale = 1.0 * width / this.plugin.desktopWidth;
-
+    var scaleFitHeight = 1.0 * height / this.plugin.desktopHeight;
+    var scaleFitWidth = 1.0 * width / this.plugin.desktopWidth;
+    var scale = Math.min(scaleFitHeight, scaleFitWidth);
     if (scale > 1.0) {
       remoting.debug.log('scale up is not supported');

[Wez, 2011/08/13 01:38:51]

Don't we need to be sure to re-set things to 1:1 here, in case scaling was
previously in effect?

[Lambros, 2011/08/13 02:13:30]

Done.

       return;
     }
 
     this.plugin.width = this.plugin.desktopWidth * scale;
     this.plugin.height = this.plugin.desktopHeight * scale;
   } else {
     remoting.debug.log('scale to fit is turned off.');
     this.plugin.width = this.plugin.desktopWidth;
     this.plugin.height = this.plugin.desktopHeight;
   }
@@ skipping 12 matching lines @@
     'video_bandwidth': this.plugin.videoBandwidth,
     'capture_latency': this.plugin.videoCaptureLatency,
     'encode_latency': this.plugin.videoEncodeLatency,
     'decode_latency': this.plugin.videoDecodeLatency,
     'render_latency': this.plugin.videoRenderLatency,
     'roundtrip_latency': this.plugin.roundTripLatency
   };
 };
 
 }());