[Webapp Refactor] Removes the client plugin on disconnect.

remoting/webapp/base/js/application.js

 // Copyright 2014 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
  * Interface abstracting the Application functionality.
  */
 
 'use strict';
@@ skipping 103 matching lines @@
 
 /**
  * Called when a new session has been connected.
  *
  * @param {remoting.ConnectionInfo} connectionInfo
  * @return {void} Nothing.
  */
 remoting.Application.prototype.onConnected = function(connectionInfo) {
   this.sessionConnectedHooks_ = new base.Disposables(
     new base.EventHook(connectionInfo.session(), 'stateChanged',
-                       this.onClientStateChange_.bind(this)),
+                       this.onSessionFinished_.bind(this)),
     new base.RepeatingTimer(this.updateStatistics_.bind(this), 1000)
   );
   remoting.clipboard.startSession();
 
   this.delegate_.handleConnected(connectionInfo);
 };
 
 /**
  * Called when the current session has been disconnected.
  *
@@ skipping 51 matching lines @@
   return this.sessionConnector_;
 };
 
 /**
  * Callback function called when the state of the client plugin changes. The
  * current and previous states are available via the |state| member variable.
  *
  * @param {remoting.ClientSession.StateEvent=} state
  * @private
  */
-remoting.Application.prototype.onClientStateChange_ = function(state) {
+remoting.Application.prototype.onSessionFinished_ = function(state) {
   switch (state.current) {
     case remoting.ClientSession.State.CLOSED:
       console.log('Connection closed by host');
       this.onDisconnected();
       break;
     case remoting.ClientSession.State.FAILED:
       var error = remoting.clientSession.getError();
       console.error('Client plugin reported connection failed: ' +
                     error.toString());
       if (error === null) {
         error = remoting.Error.unexpected();
       }
       this.onError(error);
       break;
 
     default:
       console.error('Unexpected client plugin state: ' + state.current);
       // This should only happen if the web-app and client plugin get out of
       // sync, so MISSING_PLUGIN is a suitable error.
       this.onError(new remoting.Error(remoting.Error.Tag.MISSING_PLUGIN));
       break;
   }
 
   base.dispose(this.sessionConnectedHooks_);
   this.sessionConnectedHooks_= null;
-  remoting.clientSession.dispose();
-  remoting.clientSession = null;
+  this.sessionConnector_.closeSession();
 };
 
 /** @private */
 remoting.Application.prototype.updateStatistics_ = function() {
   var perfstats = remoting.clientSession.getPerfStats();
   remoting.stats.update(perfstats);
   remoting.clientSession.logStatistics(perfstats);
 };
 
 
@@ skipping 85 matching lines @@
  * Perform any application-specific cleanup before exiting. This is called in
  * lieu of start() if the user declines the app permissions, and will usually
  * be called immediately prior to exiting, although delegates should not rely
  * on this.
  */
 remoting.Application.Delegate.prototype.handleExit = function() {};
 
 
 /** @type {remoting.Application} */
 remoting.app = null;