Index: webrtc/video/receive_statistics_proxy.cc |
diff --git a/webrtc/video/receive_statistics_proxy.cc b/webrtc/video/receive_statistics_proxy.cc |
index ff419249d4f2fe2bcb65d63115b391d5d7e2cf14..ef0920831c1b04431e12e565835e627323527792 100644 |
--- a/webrtc/video/receive_statistics_proxy.cc |
+++ b/webrtc/video/receive_statistics_proxy.cc |
@@ -77,6 +77,17 @@ void ReceiveStatisticsProxy::UpdateHistograms() { |
if (decode_ms != -1) |
RTC_LOGGED_HISTOGRAM_COUNTS_1000("WebRTC.Video.DecodeTimeInMs", decode_ms); |
+ int jb_delay_ms = jitter_buffer_delay_counter_.Avg(kMinRequiredDecodeSamples); |
+ if (jb_delay_ms != -1) { |
+ RTC_LOGGED_HISTOGRAM_COUNTS_10000("WebRTC.Video.JitterBufferDelayInMs", |
+ jb_delay_ms); |
+ } |
+ int target_delay_ms = target_delay_counter_.Avg(kMinRequiredDecodeSamples); |
stefan-webrtc
2016/04/26 08:19:26
We also have "current delay ms" which refers to th
åsapersson
2016/04/26 10:23:58
Ok, added current delay.
|
+ if (target_delay_ms != -1) { |
+ RTC_LOGGED_HISTOGRAM_COUNTS_10000("WebRTC.Video.TargetDelayInMs", |
+ target_delay_ms); |
+ } |
+ |
int delay_ms = delay_counter_.Avg(kMinRequiredDecodeSamples); |
if (delay_ms != -1) |
RTC_LOGGED_HISTOGRAM_COUNTS_10000("WebRTC.Video.OnewayDelayInMs", delay_ms); |
@@ -170,6 +181,8 @@ void ReceiveStatisticsProxy::OnDecoderTiming(int decode_ms, |
stats_.min_playout_delay_ms = min_playout_delay_ms; |
stats_.render_delay_ms = render_delay_ms; |
decode_time_counter_.Add(decode_ms); |
+ jitter_buffer_delay_counter_.Add(jitter_buffer_ms); |
+ target_delay_counter_.Add(target_delay_ms); |
åsapersson
2016/04/20 10:48:47
Let target delay stats replace metric below and th
åsapersson
2016/04/26 10:23:58
Reland CL above?
stefan-webrtc
2016/04/26 10:39:42
Sgtm if Magnus agrees.
|
// Network delay (rtt/2) + target_delay_ms (jitter delay + decode time + |
// render delay). |
delay_counter_.Add(target_delay_ms + rtt_ms / 2); |