Start logging host-version info in the Chromoting client.

remoting/webapp/crd/js/log_to_server.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
  * Module for sending log entries to the server.
  */
 
 'use strict';
@@ skipping 13 matching lines @@
   /** @private */
   this.sessionIdGenerationTime_ = 0;
   /** @private */
   this.sessionStartTime_ = new Date().getTime();
   /** @private */
   this.signalStrategy_ = signalStrategy;
   /** @private {string} */
   this.connectionType_ = '';
   /** @private */
   this.authTotalTime_ = 0;
+  /** @private {string} */
+  this.hostVersion_ = '';
 
   this.setSessionId_();
   signalStrategy.sendConnectionSetupResults(this);
 };
 
 // Constants used for generating a session ID.
 /** @private */
 remoting.LogToServer.SESSION_ID_ALPHABET_ =
     'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890';
 /** @private */
@@ skipping 127 matching lines @@
  * @private
  * @param {remoting.ServerLogEntry} entry
  */
 remoting.LogToServer.prototype.log_ = function(entry) {
   // Log the time taken to get to this point from the time this session started.
   // Exclude time taken for authorization.
   var sessionDurationInSeconds =
       (new Date().getTime() - this.sessionStartTime_ -
           this.authTotalTime_) / 1000.0;
   entry.addSessionDuration(sessionDurationInSeconds);
+  // The host-version will be blank for logs before a session has been created.
+  // For example, the signal-strategy log-entries won't have host version info.

[Jamie, 2015/04/24 20:34:02]

Is this true even for multiple connections from the same instance of the app? I
don't see the host version being explicitly cleared anywhere.

[anandc, 2015/04/24 21:36:35]

Yes. It's because a new logToServer object is created for each new session:
https://code.google.com/p/chromium/codesearch#chromium/src/remoting/webapp/cr...
and the constructor for logToServer sets it to blank.
Also confirmed by testing multiple connections, to the same machine and to
different machines, IT2Me and Me2Me, simultaneous and sequential connections,
within the same instance of the app.

+  entry.addHostVersion(this.hostVersion_);
 
   // Send the stanza to the debug log.
   console.log('Enqueueing log entry:');
   entry.toDebugLog(1);
 
   var stanza = '<cli:iq to="' + remoting.settings.DIRECTORY_BOT_JID + '" ' +
                        'type="set" xmlns:cli="jabber:client">' +
                  '<gr:log xmlns:gr="google:remoting">' +
                    entry.toStanza() +
                  '</gr:log>' +
@@ skipping 62 matching lines @@
 };
 
 /**
  * @param {number} totalTime The value of time taken to complete authorization.
  * @return {void} Nothing.
  */
 remoting.LogToServer.prototype.setAuthTotalTime = function(totalTime) {
   this.authTotalTime_ = totalTime;
 };
 
+/**
+ * @param {string} hostVersion Version of the host for current session.
+ * @return {void} Nothing.
+ */
+remoting.LogToServer.prototype.setHostVersion = function(hostVersion) {
+  this.hostVersion_ = hostVersion;
+};
+