[Webapp Refactor] Remove remoting.clientSession.

remoting/webapp/crd/js/client_plugin_impl.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 that wraps low-level details of interacting with the client plugin.
  *
  * This abstracts a <embed> element and controls the plugin which does
  * the actual remoting work. It also handles differences between
@@ skipping 293 matching lines @@
     base.getNumberAttr(message.data, 'renderLatency');
     base.getNumberAttr(message.data, 'roundtripLatency');
     this.perfStats_ =
         /** @type {remoting.ClientSession.PerfStats} */ (message.data);
 
   } else if (message.method == 'injectClipboardItem') {
     var mimetype = base.getStringAttr(message.data, 'mimeType');
     var item = base.getStringAttr(message.data, 'item');
     this.updateClipboardData_(mimetype, item);
 
-  } else if (message.method == 'onFirstFrameReceived') {

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

Removed as not used.

[Jamie, 2015/04/14 23:52:47]

This will be used soon to log the amount of time before the first frame is
received. I think we need to keep it.

[kelvinp, 2015/04/15 00:30:21]

Good catch. Resurrected!

-    if (remoting.clientSession) {
-      remoting.clientSession.onFirstFrameReceived();
-    }
-
   } else if (message.method == 'fetchPin') {
     // The pairingSupported value in the dictionary indicates whether both
     // client and host support pairing. If the client doesn't support pairing,
     // then the value won't be there at all, so give it a default of false.
     var pairingSupported = base.getBooleanAttr(message.data, 'pairingSupported',
                                           false);
     this.credentials_.getPIN(pairingSupported).then(
         this.onPinFetched_.bind(this)
     );
 
@@ skipping 542 matching lines @@
 remoting.DefaultClientPluginFactory.prototype.preloadPlugin = function() {
   if (remoting.settings.CLIENT_PLUGIN_TYPE != 'pnacl') {
     return;
   }
 
   var plugin = remoting.ClientPluginImpl.createPluginElement_();
   plugin.addEventListener(
       'loadend', function() { document.body.removeChild(plugin); }, false);
   document.body.appendChild(plugin);
 };