Allow full-screen mode when not connected, and remember the previous setting.

remoting/webapp/crd/js/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.
  *
  * The ClientSession class controls lifetime of the client plugin
  * object and provides the plugin with the functionality it needs to
@@ skipping 593 matching lines @@
 remoting.ClientSession.prototype.removePlugin = function() {
   if (this.plugin_) {
     this.plugin_.element().removeEventListener(
         'focus', this.callPluginGotFocus_, false);
     this.plugin_.element().removeEventListener(
         'blur', this.callPluginLostFocus_, false);
     this.plugin_.dispose();
     this.plugin_ = null;
   }
 
-  // Leave full-screen mode, and stop listening for related events.
-  var listener = this.callOnFullScreenChanged_;
-  remoting.fullscreen.activate(
-      false,
-      function() {
-        remoting.fullscreen.removeListener(listener);
-      });
+  // Stop listening for full-screen events.
+  remoting.fullscreen.removeListener(this.callOnFullScreenChanged_);

[kelvinp, 2015/01/23 23:39:58]

Rant: It is weird that sometimes the clientSession hooks its own fullscreen
listener like here, but sometimes via a proxy remoting.onResize().
IMHO, if clientSession needs to be shared among LG (which does't care about
fullscreen)  and chromoting.  It will be better to have the owning object of
client session to handle the event hooking.

[Jamie, 2015/01/24 00:46:35]

We are going to have to disentangle full-screen from this class, but we only
need it for bump-scroll support (which is also not used for app-remoting), so I
think we'll move both together in some clever way that Gary will come up with :)

   this.updateClientSessionUi_(null);
 
   this.container_.removeEventListener('mousemove',
                                       this.updateMouseCursorPosition_,
                                       true);
 };
 
 /**
  * @param {remoting.ClientSession} clientSession The active session, or null if
  *     there is no connection.
@@ skipping 1095 matching lines @@
  * @param {Object} message The parsed extension message data.
  * @return {boolean} True if the message was recognized, false otherwise.
  */
 remoting.ClientSession.prototype.handleExtensionMessage =
     function(type, message) {
   if (this.videoFrameRecorder_) {
     return this.videoFrameRecorder_.handleMessage(type, message);
   }
   return false;
 }