CRD Viewport Management refactor

remoting/webapp/crd/js/host.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
  * The deserialized form of the chromoting host as returned by Apiary.
  */
 
 'use strict';
@@ skipping 37 matching lines @@
 
 /**
  * @constructor
  * @param {string} hostId
  * @struct
  */
 remoting.Host.Options = function(hostId) {
   /** @private */
   this.hostId_ = hostId;
   /** @type {boolean} */
-  this.shrinkToFit = false;
+  this.shrinkToFit = true;
   /** @type {boolean} */
-  this.resizeToClient = false;
+  this.resizeToClient = true;
   /** @type {string} */
   this.remapKeys = '';
   /** @type {number} */
   this.desktopScale = 1;
 };
 
 remoting.Host.Options.prototype.save = function() {
   // TODO(kelvinp): Migrate pairingInfo to use this class as well and get rid of
   // remoting.HostSettings.
   remoting.HostSettings.save(this.hostId_, this);
 };
 
 
 /** @return {Promise} A promise that resolves when the settings are loaded. */
 remoting.Host.Options.prototype.load = function() {
   var that = this;
   return base.Promise.as(remoting.HostSettings.load, [this.hostId_]).then(
     /**
      * @param {Object.<string|boolean|number>} options
      */
     function(options) {
+      // Must be defaulted to true so that app-remoting can resize the host
+      // upon launching.
+      // TODO(kelvinp): Uses a separate host options for app-remoting that
+      // hardcodes resizeToClient to true.
       that.resizeToClient =
-          getBooleanAttr(options, 'resizeToClient', false);
-      that.shrinkToFit = getBooleanAttr(options, 'shrinkToFit', false);
+          getBooleanAttr(options, 'resizeToClient', true);
+      that.shrinkToFit = getBooleanAttr(options, 'shrinkToFit', true);
       that.desktopScale = getNumberAttr(options, 'desktopScale', 1);
       that.remapKeys = getStringAttr(options, 'remapKeys', '');
     });
 };
 
 /**
  * Determine whether a host needs to be manually updated. This is the case if
  * the host's major version number is more than 1 lower than that of the web-
  * app (a difference of 1 is tolerated due to the different update mechanisms)
  * and if the host is on-line (off-line hosts are not expected to auto-update).
@@ skipping 11 matching lines @@
   var hostMajorVersion = parseInt(host.hostVersion, 10);
   if (isNaN(hostMajorVersion)) {
     // Host versions 26 and higher include the version number in heartbeats,
     // so if it's missing then the host is at most version 25.
     hostMajorVersion = 25;
   }
   return (parseInt(webappVersion, 10) - hostMajorVersion) > 1;
 };
 
 })();