Include time elapsed since start of a remoting session in each log entry.

remoting/webapp/crd/js/desktop_viewport.js

 // Copyright 2015 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
  * Provides view port management utilities below for a desktop remoting session.
  * - Enabling bump scrolling
  * - Resizing the viewport to fit the host desktop
  * - Resizing the host desktop to fit the client viewport.
@@ skipping 323 matching lines @@
       remoting.fullscreen.isActive(), this.hostOptions_.shrinkToFit);
 
   // Resize the plugin if necessary.
   console.log('plugin dimensions:' + newSize.width + 'x' + newSize.height);
   this.pluginElement_.style.width = newSize.width + 'px';
   this.pluginElement_.style.height = newSize.height + 'px';
 
   // When we receive the first plugin dimensions from the host, we know that
   // remote host has started.
   remoting.app.onVideoStreamingStarted();
+
+  remoting.clientSession.logToServer.logTotalTimeForInitialConnection();

[Jamie, 2015/02/25 21:12:35]

I would prefer that we don't introduce a dependency on remoting.clientSession in
this class. Doing this logging in onVideoStreamingStarted would seem more
appropriate. Kelvin can probably chime in here as to whether or not that's a
good idea.

In any case, you shouldn't call this multiple times, which will happen with this
layout, since I think updateDimensions is called whenever the remote desktop
size changes. An alternative would be to hook the connected event directly;
you'd lose information as to whether or not the video stream works, but that
feels orthogonal to the time taken to connect.

[anandc, 2015/02/26 20:21:49]

The user perceives a connection has been completed only when video-streaming
begins. As discussed offline, I'll create a new session-state for that, in a
separate CL.

+
 };
 
 /**
  * Helper function accepting client and host dimensions, and returning a chosen
  * size for the plugin element, in DIPs.
  *
  * @param {{width: number, height: number}} clientSizeDips Available client
  *     dimensions, in DIPs.
  * @param {number} clientPixelRatio Number of physical pixels per client DIP.
  * @param {{width: number, height: number}} desktopSize Size of the host desktop
@@ skipping 122 matching lines @@
   return { width: pluginWidth, height: pluginHeight };
 };
 
 /** @private */
 remoting.DesktopViewport.prototype.resetScroll_ = function() {
   this.pluginContainer_.style.marginTop = '0px';
   this.pluginContainer_.style.marginLeft = '0px';
 };
 
 }());