From 03bbef5e1f40413667826d6a8006abf7a0b630b6 Mon Sep 17 00:00:00 2001 From: Johannes Kron Date: Fri, 20 Sep 2019 17:27:31 +0200 Subject: [PATCH] Fix accidental change of transport time metric The transport time metric was accidentially changed by the CL https://webrtc-review.googlesource.com/c/src/+/153660 This CL restore the transport time metric to how it has been measured before, that is, time from encoder output to decoder input. Bug: webrtc:10975 Change-Id: I66f022f26976451d28c0374b22849f14f9c02378 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/153886 Reviewed-by: Artem Titov Commit-Queue: Johannes Kron Cr-Commit-Position: refs/heads/master@{#29259} --- test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc index abff61da95..cd3a95d075 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc @@ -548,7 +548,7 @@ void DefaultVideoQualityAnalyzer::ProcessComparison( *comparison.frame_stats.rendered_frame_width * *comparison.frame_stats.rendered_frame_height); stats->transport_time_ms.AddSample( - (frame_stats.received_time - frame_stats.encoded_time).ms()); + (frame_stats.decode_start_time - frame_stats.encoded_time).ms()); stats->total_delay_incl_transport_ms.AddSample( (frame_stats.rendered_time - frame_stats.captured_time).ms()); stats->decode_time_ms.AddSample(