Initialize remoting.ClientSession.PerfStats with zeros

remoting/webapp/base/js/client_session.js

Index: remoting/webapp/base/js/client_session.js
diff --git a/remoting/webapp/base/js/client_session.js b/remoting/webapp/base/js/client_session.js
index eda42d7891f85299f911988dd9d3cb5ee8bf005b..9c8b4756fd31bec7e5221b454c04ab09f16db6c1 100644
--- a/remoting/webapp/base/js/client_session.js
+++ b/remoting/webapp/base/js/client_session.js
@@ -202,30 +202,30 @@ remoting.ClientSession.ConnectionError.fromString = function(error) {
  * @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';