Report max interframe delay over window insdead of interframe delay sum

webrtc/video/receive_statistics_proxy.h

Index: webrtc/video/receive_statistics_proxy.h
diff --git a/webrtc/video/receive_statistics_proxy.h b/webrtc/video/receive_statistics_proxy.h
index 0cd0b529aa3b5e2f8cd8a660909359578578476f..21d0657fd5d6975cc97b8ae0577784959b53c6a8 100644
--- a/webrtc/video/receive_statistics_proxy.h
+++ b/webrtc/video/receive_statistics_proxy.h
@@ -19,6 +19,7 @@
 #include "webrtc/common_video/include/frame_callback.h"
 #include "webrtc/modules/video_coding/include/video_coding_defines.h"
 #include "webrtc/rtc_base/criticalsection.h"
+#include "webrtc/rtc_base/moving_max_counter.h"
 #include "webrtc/rtc_base/rate_statistics.h"
 #include "webrtc/rtc_base/ratetracker.h"
 #include "webrtc/rtc_base/thread_annotations.h"
@@ -152,6 +153,7 @@ class ReceiveStatisticsProxy : public VCMReceiveStatisticsCallback,
   int64_t e2e_delay_max_ms_screenshare_ GUARDED_BY(crit_);
   int64_t interframe_delay_max_ms_video_  GUARDED_BY(crit_);
   int64_t interframe_delay_max_ms_screenshare_  GUARDED_BY(crit_);
+  rtc::MovingMaxCounter<int> interframe_delay_max_moving_ GUARDED_BY(crit_);

[kwiberg-webrtc, 2017/08/22 21:40:47]

What if you made this one mutable instead, and made MovingMaxCounter::Max()
non-const?

This class uses the counter responsibly---always passing the current time,
calling Max() but not Add() from const methods---so its callers will not observe
any const conventions being broken.

[ilnik, 2017/08/23 07:46:50]

Yes, this is a great idea. Done it.

   MaxCounter freq_offset_counter_ GUARDED_BY(crit_);
   int64_t first_report_block_time_ms_ GUARDED_BY(crit_);
   ReportBlockStats report_block_stats_ GUARDED_BY(crit_);