zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[Stats] Remove jitterBufferDelay TODO; it's already implemented.

Browse Source 
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 <hbos@webrtc.org>
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30485}
This commit is contained in:
Henrik Boström
2020-02-07 16:12:16 +01:00
committed by Commit Bot
parent 8e8b36a94a
commit 189849fa0f
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
api/stats/rtcstats_objects.h
View File

@ -294,9 +294,6 @@ class RTC_EXPORT RTCMediaStreamTrackStats final : public RTCStats {
RTCStatsMember<bool> detached;
// See |RTCMediaStreamTrackKind| for valid values.
RTCStatsMember<std::string> kind;
// TODO(gustaf): Implement jitter_buffer_delay for video (currently
// implemented for audio only).
// https://crbug.com/webrtc/8318
RTCStatsMember<double> jitter_buffer_delay;
RTCStatsMember<uint64_t> jitter_buffer_emitted_count;
// Video-only members