Index: remoting/webapp/base/js/session_logger_unittest.js |
diff --git a/remoting/webapp/base/js/session_logger_unittest.js b/remoting/webapp/base/js/session_logger_unittest.js |
index 460dec55ddd12d4383d8c6675f50a12121edf2d3..c1d55b45716e77476d8318287ad8519aab4b4b19 100644 |
--- a/remoting/webapp/base/js/session_logger_unittest.js |
+++ b/remoting/webapp/base/js/session_logger_unittest.js |
@@ -277,36 +277,19 @@ QUnit.test('logStatistics()', function(assert) { |
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, { |
@@ -323,12 +306,17 @@ QUnit.test('logStatistics()', function(assert) { |
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 |
}); |
}); |
@@ -349,17 +337,6 @@ QUnit.test('logStatistics() should not log if all stats are zeros ', |
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); |
}); |