[Webapp Refactor] Remove remoting.clientSession.

remoting/webapp/app_remoting/js/app_remoting.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
  * This class implements the functionality that is specific to application
  * remoting ("AppRemoting" or AR).
  */
 
@@ skipping 75 matching lines @@
 
 /**
  * @return {string} Application product name to be used in UI.
  * @override {remoting.ApplicationInterface}
  */
 remoting.AppRemoting.prototype.getApplicationName = function() {
   var manifest = chrome.runtime.getManifest();
   return manifest.name;
 };
 
+remoting.AppRemoting.prototype.getActivity = function() {

[kelvinp, 2015/04/14 23:16:32]

This will be fixed to expose the correct activity in CL
https://codereview.chromium.org/1082973002/

+  return null;
+};
+
 /**
  * @param {!remoting.Error} error The failure reason.
  * @override {remoting.ApplicationInterface}
  */
 remoting.AppRemoting.prototype.signInFailed_ = function(error) {
   this.onError(error);
 };
 
 /**
  * @override {remoting.ApplicationInterface}
@@ skipping 42 matching lines @@
           response.host.hostId) {
         var hostJid = response.hostJid;
         that.contextMenu_.setHostId(response.host.hostId);
         var host = new remoting.Host(response.host.hostId);
         host.jabberId = hostJid;
         host.authorizationCode = response.authorizationCode;
         host.sharedSecret = response.sharedSecret;
 
         remoting.setMode(remoting.AppMode.CLIENT_CONNECTING);
 
-        var idleDetector = new remoting.IdleDetector(
-            document.getElementById('idle-dialog'),
-            remoting.app.disconnect.bind(remoting.app));
-
         /**
          * @param {string} tokenUrl Token-issue URL received from the host.
          * @param {string} hostPublicKey Host public key (DER and Base64
          *     encoded).
          * @param {string} scope OAuth scope to request the token for.
          * @param {function(string, string):void} onThirdPartyTokenFetched
          *     Callback.
          */
         var fetchThirdPartyToken = function(
             tokenUrl, hostPublicKey, scope, onThirdPartyTokenFetched) {
@@ skipping 33 matching lines @@
 
 /**
  * @param {remoting.ConnectionInfo} connectionInfo
  */
 remoting.AppRemoting.prototype.onConnected = function(connectionInfo) {
   this.supportsGoogleDrive_ = connectionInfo.session().hasCapability(
       remoting.ClientSession.Capability.GOOGLE_DRIVE);
 
   connectionInfo.plugin().extensions().register(this);
 
+  var session = connectionInfo.session();
+  var idleDetector = new remoting.IdleDetector(

[kelvinp, 2015/04/14 23:16:32]

The IdleDetector should be started only when the session is connected.

+      document.getElementById('idle-dialog'),
+      session.disconnect.bind(session, remoting.Error.none()));
+
   this.connectedView_ = new remoting.AppConnectedView(
       document.getElementById('client-container'), connectionInfo);
 
   // Map Cmd to Ctrl on Mac since hosts typically use Ctrl for keyboard
   // shortcuts, but we want them to act as natively as possible.
   if (remoting.platformIsMac()) {
     connectionInfo.plugin().setRemapKeys('0x0700e3>0x0700e0,0x0700e7>0x0700e4');
   }
 };
 
@@ skipping 149 matching lines @@
 };
 
 /**
  * @return {string}
  * @private
  */
 remoting.AppRemoting.prototype.runApplicationUrl_ = function() {
   return remoting.settings.APP_REMOTING_API_BASE_URL + '/applications/' +
       remoting.settings.getAppRemotingApplicationId() + '/run';
 };