Initialize remoting.ClientSession.PerfStats with zeros

remoting/webapp/base/js/client_session.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 handling creation and teardown of a remoting client session.
  *
  * The ClientSession class controls lifetime of the client plugin
  * object and provides the plugin with the functionality it needs to
@@ skipping 184 matching lines @@
     return remoting.ClientSession.ConnectionError.UNKNOWN;
   }
   return remoting.ClientSession.ConnectionError[error];
 }
 
 /**
  * Type used for performance statistics collected by the plugin.
  * @constructor
  */
 remoting.ClientSession.PerfStats = function() {};
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.videoBandwidth;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.videoFrameRate;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.captureLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.maxCaptureLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.encodeLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.maxEncodeLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.decodeLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.maxDecodeLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.renderLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.maxRenderLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.roundtripLatency;
-/** @type {number} */
-remoting.ClientSession.PerfStats.prototype.maxRoundtripLatency;
+/** @type {!number} */

[Jamie, 2016/05/23 18:18:27]

Number is not a nullable type, so you don't need this (undefined and null are
separate types in JS).

[Yuwei, 2016/05/23 18:25:23]

Done.

+remoting.ClientSession.PerfStats.prototype.videoBandwidth = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.videoFrameRate = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.captureLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.maxCaptureLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.encodeLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.maxEncodeLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.decodeLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.maxDecodeLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.renderLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.maxRenderLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.roundtripLatency = 0;
+/** @type {!number} */
+remoting.ClientSession.PerfStats.prototype.maxRoundtripLatency = 0;
 
 // Keys for connection statistics.
 remoting.ClientSession.STATS_KEY_VIDEO_BANDWIDTH = 'videoBandwidth';
 remoting.ClientSession.STATS_KEY_VIDEO_FRAME_RATE = 'videoFrameRate';
 remoting.ClientSession.STATS_KEY_CAPTURE_LATENCY = 'captureLatency';
 remoting.ClientSession.STATS_KEY_ENCODE_LATENCY = 'encodeLatency';
 remoting.ClientSession.STATS_KEY_DECODE_LATENCY = 'decodeLatency';
 remoting.ClientSession.STATS_KEY_RENDER_LATENCY = 'renderLatency';
 remoting.ClientSession.STATS_KEY_ROUNDTRIP_LATENCY = 'roundtripLatency';
 
@@ skipping 474 matching lines @@
   } else if (previous == State.CONNECTED && current == State.FAILED) {
     return State.CONNECTION_DROPPED;
   }
   return current;
 };
 
 /** @private */
 remoting.ClientSession.prototype.reportStatistics = function() {
   this.logger_.logStatistics(this.plugin_.getPerfStats());
 };