DescriptionChange reporting of timing frames conditions in GetStats on receive side
Instead of the longest frame since the last GetStats call, the longest
frame received during last 10 seconds should be returned in GetStats().
Previous way is not a good idea because there are maybe several
consumers of GetStats calls. If not all of them parse timing frame
reports, some of them may be lost.
Also, streamline reporting of TimingFrames via GetStats (remove separate
methods and use VideoReceiveStream::Stats struct instead).
BUG=webrtc:7594
Review-Url: https://codereview.webrtc.org/3008983002
Cr-Commit-Position: refs/heads/master@{#19650}
Committed: https://chromium.googlesource.com/external/webrtc/+/75204c5ccdda561ccffbfa3a8fa94feac8e19e30
Patch Set 1 #Patch Set 2 : Fix timing frames ToString implementation #
Total comments: 4
Patch Set 3 : Implement Sprang@ comments #
Total comments: 6
Patch Set 4 : Implement Tommi@ comments #
Messages
Total messages: 28 (17 generated)
|