From 189849fa0feca530781aa20f95de93539bbe12f0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Henrik=20Bostr=C3=B6m?= Date: Fri, 7 Feb 2020 16:12:16 +0100 Subject: [PATCH] [Stats] Remove jitterBufferDelay TODO; it's already implemented. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This TODO says this metric is only available for audio and should also be implemented for video, but ever since M76 this has been implemented for both audio and video (https://crbug.com/webrtc/10450). TBR=guido@webrtc.org, hta@webrtc.org NOTRY=True Bug: webrtc:10450 Change-Id: Icf2b60fdacae606c66f9d03492f107df9e32ba33 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/168343 Reviewed-by: Henrik Boström Commit-Queue: Henrik Boström Cr-Commit-Position: refs/heads/master@{#30485} --- api/stats/rtcstats_objects.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/api/stats/rtcstats_objects.h b/api/stats/rtcstats_objects.h index af91a85aed..43f8839b90 100644 --- a/api/stats/rtcstats_objects.h +++ b/api/stats/rtcstats_objects.h @@ -294,9 +294,6 @@ class RTC_EXPORT RTCMediaStreamTrackStats final : public RTCStats { RTCStatsMember detached; // See |RTCMediaStreamTrackKind| for valid values. RTCStatsMember kind; - // TODO(gustaf): Implement jitter_buffer_delay for video (currently - // implemented for audio only). - // https://crbug.com/webrtc/8318 RTCStatsMember jitter_buffer_delay; RTCStatsMember jitter_buffer_emitted_count; // Video-only members