From 61bd5d9c1c05f24044918f538b95646e9523c167 Mon Sep 17 00:00:00 2001 From: Di Wu Date: Tue, 27 Jul 2021 22:21:22 -0700 Subject: [PATCH] Video jitter stats fix: Convert RTP timestamp MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit stats.rtp_stats.jitter is a RTP timestamp so we needed to convert it back to regular timestamps See https://bugs.chromium.org/p/webrtc/issues/detail?id=12980#c7 Bug: webrtc:12980 Change-Id: I0d94a22e043ac6ecec4926d950abbdcf787b7168 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/227100 Reviewed-by: Henrik Boström Reviewed-by: Erik Språng Commit-Queue: Di Wu Cr-Commit-Position: refs/heads/master@{#34590} --- media/engine/webrtc_video_engine.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/media/engine/webrtc_video_engine.cc b/media/engine/webrtc_video_engine.cc index d42047aa6c..8376b01735 100644 --- a/media/engine/webrtc_video_engine.cc +++ b/media/engine/webrtc_video_engine.cc @@ -3161,7 +3161,7 @@ WebRtcVideoChannel::WebRtcVideoReceiveStream::GetVideoReceiverInfo( stats.rtp_stats.packet_counter.padding_bytes; info.packets_rcvd = stats.rtp_stats.packet_counter.packets; info.packets_lost = stats.rtp_stats.packets_lost; - info.jitter_ms = stats.rtp_stats.jitter; + info.jitter_ms = stats.rtp_stats.jitter / (kVideoCodecClockrate / 1000); info.framerate_rcvd = stats.network_frame_rate; info.framerate_decoded = stats.decode_frame_rate;