The chromoting client logs connection statistics to the server.

remoting/webapp/me2mom/client_session.js

 // Copyright (c) 2011 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 handling creation and teardown of a remoting client session.
  *
  * This abstracts a <embed> element and controls the plugin which does the
  * actual remoting work.  There should be no UI code inside this class.  It
@@ skipping 50 matching lines @@
 
 /** @enum {number} */
 remoting.ClientSession.ConnectionError = {
   NONE: 0,
   HOST_IS_OFFLINE: 1,
   SESSION_REJECTED: 2,
   INCOMPATIBLE_PROTOCOL: 3,
   NETWORK_FAILURE: 4
 };
 
+// Keys for connection statistics.
+remoting.ClientSession.STATS_KEY_VIDEO_BANDWIDTH = 'video_bandwidth';
+remoting.ClientSession.STATS_KEY_VIDEO_FRAME_RATE = 'video_frame_rate';
+remoting.ClientSession.STATS_KEY_CAPTURE_LATENCY = 'capture_latency';
+remoting.ClientSession.STATS_KEY_ENCODE_LATENCY = 'encode_latency';
+remoting.ClientSession.STATS_KEY_DECODE_LATENCY = 'decode_latency';
+remoting.ClientSession.STATS_KEY_RENDER_LATENCY = 'render_latency';
+remoting.ClientSession.STATS_KEY_ROUNDTRIP_LATENCY = 'roundtrip_latency';
+
 /**
  * The current state of the session.
  * @type {remoting.ClientSession.State}
  */
 remoting.ClientSession.prototype.state = remoting.ClientSession.State.UNKNOWN;
 
 /**
  * The last connection error. Set when state is set to CONNECTION_FAILED.
  * @type {remoting.ClientSession.ConnectionError}
  */
@@ skipping 325 matching lines @@
                      this.plugin.width + 'x' + this.plugin.height + '.');
   this.plugin.setScaleToFit(remoting.scaleToFit);
 };
 
 /**
  * Returns an associative array with a set of stats for this connection.
  *
  * @return {Object.<string, number>} The connection statistics.
  */
 remoting.ClientSession.prototype.stats = function() {
-  return {
-    'video_bandwidth': this.plugin.videoBandwidth,
-    'video_frame_rate': this.plugin.videoFrameRate,
-    'capture_latency': this.plugin.videoCaptureLatency,
-    'encode_latency': this.plugin.videoEncodeLatency,
-    'decode_latency': this.plugin.videoDecodeLatency,
-    'render_latency': this.plugin.videoRenderLatency,
-    'roundtrip_latency': this.plugin.roundTripLatency
-  };
+  var dict = {};
+  dict[remoting.ClientSession.STATS_KEY_VIDEO_BANDWIDTH] =
+      this.plugin.videoBandwidth;
+  dict[remoting.ClientSession.STATS_KEY_VIDEO_FRAME_RATE] =
+      this.plugin.videoFrameRate;
+  dict[remoting.ClientSession.STATS_KEY_CAPTURE_LATENCY] =
+      this.plugin.videoCaptureLatency;
+  dict[remoting.ClientSession.STATS_KEY_ENCODE_LATENCY] =
+      this.plugin.videoEncodeLatency;
+  dict[remoting.ClientSession.STATS_KEY_DECODE_LATENCY] =
+      this.plugin.videoDecodeLatency;
+  dict[remoting.ClientSession.STATS_KEY_RENDER_LATENCY] =
+      this.plugin.videoRenderLatency;
+  dict[remoting.ClientSession.STATS_KEY_ROUNDTRIP_LATENCY] =
+      this.plugin.roundTripLatency;
+  return dict;

[Jamie, 2011/12/09 19:47:56]

Nit: Why change to this form of construction?

[simonmorris, 2011/12/09 21:12:12]

The other form only works with compile-time
constants.

 };
+
+/**
+ * Logs statistics.
+ *
+ * @param {Object.<string, number>} stats
+ */
+remoting.ClientSession.prototype.logStatistics = function(stats) {
+  this.logToServer.logStatistics(stats);
+};