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

remoting/webapp/crd/js/log_to_server.js

Index: remoting/webapp/crd/js/log_to_server.js
diff --git a/remoting/webapp/crd/js/log_to_server.js b/remoting/webapp/crd/js/log_to_server.js
index 8b87f1b5c4efc7273b3cb4ce5f43f36ae5926294..70d1617511e1913aa07822f0d3685538cd2edc36 100644
--- a/remoting/webapp/crd/js/log_to_server.js
+++ b/remoting/webapp/crd/js/log_to_server.js
@@ -66,6 +66,8 @@ remoting.LogToServer.prototype.logClientSessionStateChange =
       this.sessionStartTime_ = new Date().getTime();
     }
   }
+  var elapsedTimeInMs = new Date().getTime() - this.sessionStartTime_;
+
   // Log the session state change.
   var entry = remoting.ServerLogEntry.makeClientSessionStateChange(
       state, connectionError, this.mode_);
@@ -82,6 +84,9 @@ remoting.LogToServer.prototype.logClientSessionStateChange =
       this.sessionStartTime_ = 0;
     }
   }
+
+  // Log the time taken to get to this state from the time this session started.
+  entry.addElapsedTimeMs(elapsedTimeInMs);

[Jamie, 2015/02/27 02:24:46]

This is still only being added for session state change events. We need it for
signal strategy events at least, and I think there's value in having it for all
events. I haven't looked in detail, but I think you'll need to add it to the
log_() method, and look into how the session start time is set. You need to make
sure it's set before the first log message is sent, but if the LogToServer
instance is re-used between connections, you'll also need to reset it--it's
probably better to make sure the LogToServer is not re-used.

[anandc, 2015/02/27 21:49:29]

Done.

LogToServer is reset at the start of each session. See:
https://code.google.com/p/chromium/codesearch#chromium/src/remoting/webapp/cr...

Session start time is being set when we first get a "Connecting" event. See line
65 above.

The first call to log_ is made right after the first "Connecting" event. I
confirmed this by checking values in the console debugger. 

Based on all that, adding the elapsed-time field within log_ now.

   this.log_(entry);
   // Don't accumulate connection statistics across state changes.
   this.logAccumulatedStatistics_();
@@ -104,13 +109,12 @@ remoting.LogToServer.prototype.setConnectionType = function(connectionType) {
 /**
  * @param {remoting.SignalStrategy.Type} strategyType
  * @param {remoting.FallbackSignalStrategy.Progress} progress
- * @param {number} elapsedTimeInMs
  */
 remoting.LogToServer.prototype.logSignalStrategyProgress =
-    function(strategyType, progress, elapsedTimeInMs) {
+    function(strategyType, progress) {
   this.maybeExpireSessionId_();
   var entry = remoting.ServerLogEntry.makeSignalStrategyProgress(
-      this.sessionId_, strategyType, progress, elapsedTimeInMs);
+      this.sessionId_, strategyType, progress);
   this.log_(entry);
 };