Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(487)

Unified Diff: remoting/webapp/crd/js/host.js

Issue 918783002: CRD Viewport Management refactor (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reviewer's feedback Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/webapp/crd/js/host.js
diff --git a/remoting/webapp/crd/js/host.js b/remoting/webapp/crd/js/host.js
index f43d69c377da650d1c4987bf54b6ebc547ebfa5a..59129b1dfc2f3f0cbcd56e287e9b8b1a3806e040 100644
--- a/remoting/webapp/crd/js/host.js
+++ b/remoting/webapp/crd/js/host.js
@@ -55,9 +55,9 @@ 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} */
@@ -79,9 +79,13 @@ remoting.Host.Options.prototype.load = function() {
* @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', '');
});
« no previous file with comments | « remoting/webapp/crd/js/desktop_viewport.js ('k') | remoting/webapp/unittests/desktop_connected_view_unittest.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698