Stops accumulating performance statistics on the Chrome App.

remoting/webapp/base/js/session_logger_unittest.js

 // Copyright 2015 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.
 
 (function() {
 
 'use strict';
 
 /** @type {remoting.SessionLogger} */
 var logger = null;
@@ skipping 259 matching lines @@
 QUnit.test('logStatistics()', function(assert) {
   var clock = sinon.useFakeTimers();
   var Event = remoting.ChromotingEvent;
 
   // Creates the logger.
   logger = new remoting.SessionLogger(Event.Role.CLIENT, logWriter);
   logger.setLogEntryMode(Event.Mode.LGAPP);
   logger.setConnectionType('direct');
   logger.setHost(fakeHost);
 
-  // Log the statistics.
-  logger.logStatistics({
-    videoBandwidth: 1.0,
-    captureLatency: 1.0,
-    encodeLatency: 1.0,
-    decodeLatency: 0.0,
-    renderLatency: 1.0,
-    roundtripLatency: 1.0
-  });
-
-  logger.logStatistics({
-    videoBandwidth: 2.0,
-    captureLatency: 2.0,
-    encodeLatency: 1.0,
-    decodeLatency: 0.0,
-    renderLatency: 2.0,
-    roundtripLatency: 2.0
-  });
-
-  sinon.assert.notCalled(logWriterSpy);
-  // Stats should only be accumulated at |CONNECTION_STATS_ACCUMULATE_TIME|.
-  clock.tick(remoting.SessionLogger.CONNECTION_STATS_ACCUMULATE_TIME + 10);
-
+  // Log the statistics
   logger.logStatistics({
     videoBandwidth: 3.0,
-    captureLatency: 3.0,
-    encodeLatency: 1.0,
-    decodeLatency: 0.0,
-    renderLatency: 0.0,
-    roundtripLatency: 0.0
+    captureLatency: 1.0,
+    maxCaptureLatency: 3.0,
+    encodeLatency: 2.0,
+    maxEncodeLatency: 4.0,
+    decodeLatency: 3.0,
+    maxDecodeLatency: 5.0,
+    renderLatency: 4.0,
+    maxRenderLatency: 6.0,
+    roundtripLatency: 5.0,
+    maxRoundtripLatency: 7.0
   });
 
   verifyEvent(assert, 0, {
     type: Event.Type.CONNECTION_STATISTICS,
     os: Event.Os.MAC,
     os_version: '10.9.5',
     cpu: 'Intel',
     browser_version: '43.0.2357.81',
     application_id: 'extensionId',
     role: Event.Role.CLIENT,
     mode: Event.Mode.LGAPP,
     connection_type: Event.ConnectionType.DIRECT,
     host_version: 'host_version',
     host_os: remoting.ChromotingEvent.Os.OTHER,
     host_os_version: 'host_os_version',
     session_id: logger.getSessionId(),
-    video_bandwidth: 2.0,
-    capture_latency: 2.0,
-    encode_latency: 1.0,
-    decode_latency: 0.0,
-    render_latency: 1.0,
-    roundtrip_latency: 1.0
+    video_bandwidth: 3.0,
+    capture_latency: 1.0,
+    encode_latency: 2.0,
+    decode_latency: 3.0,
+    render_latency: 4.0,
+    roundtrip_latency: 5.0,
+    max_capture_latency: 3.0,
+    max_encode_latency: 4.0,
+    max_decode_latency: 5.0,
+    max_render_latency: 6.0,
+    max_roundtrip_latency: 7.0
   });
 });
 
 QUnit.test('logStatistics() should not log if all stats are zeros ',
     function(assert) {
   var clock = sinon.useFakeTimers();
   var Event = remoting.ChromotingEvent;
 
   // Creates the logger.
   logger = new remoting.SessionLogger(Event.Role.CLIENT, logWriter);
 
   logger.logStatistics({
     videoBandwidth: 0.0,
     captureLatency: 0.0,
     encodeLatency: 0.0,
     decodeLatency: 0.0,
     renderLatency: 0.0,
     roundtripLatency: 0.0
   });
 
-  clock.tick(remoting.SessionLogger.CONNECTION_STATS_ACCUMULATE_TIME + 10);
-
-  logger.logStatistics({
-    videoBandwidth: 0.0,
-    captureLatency: 0.0,
-    encodeLatency: 0.0,
-    decodeLatency: 0.0,
-    renderLatency: 0.0,
-    roundtripLatency: 0.0
-  });
-
   sinon.assert.notCalled(logWriterSpy);
 
 });
 
 QUnit.test('incrementFeatureUsage() and flushFeatureTracker()',
     function(assert) {
   var Event = remoting.ChromotingEvent;
 
   logger = new remoting.SessionLogger(Event.Role.CLIENT, logWriter);
 
@@ skipping 17 matching lines @@
   sinon.assert.notCalled(logWriterSpy);
 
   logger.flushFeatureTracker();
   verifyEvent(assert, 0, {
     type: Event.Type.FEATURE_TRACKING,
     feature_tracker: {fullscreen_esc_count: 0}
   });
 });
 
 })();