commit a57711c941b89cd1b0f666f35b053bb5b796ced4
author Erik Språng <sprang@webrtc.org> Wed Jul 24 10:47:20 2019 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Jul 24 11:24:57 2019 +0000
tree 2709707de3e5af75b2b1aae788b2b16207d11a18
parent 54d9602fe15d434dffa9539e14f7b982bd64704e

Fix issue with TransmissionOffset using new pacer code path

This CL fixes two issues related to the TransmissionOffset header
extension and the new (not yet active) pacer mode.

Previously capture time (if unset) would be populated when put into the
packet history before entering the pacer. Since the pacer now owns the
packets, this does not occur until packet is actually sent, if at all.
Capture has really nothing to do with the packet history, this should
be set by the RtpSender pre-pacing instead.

Furthermore, for retransmissions the old path would take the capture
time from the original packet, build the RTX-wrapped retransmission and
set the toffset extension of the RTX packet using that captured capture
time. Since RTX packets are now fully built before the pacer, this does
not work, and we need to transfer the capture time from the original to
the RTX packet instead.

Bug: webrtc:10633
Change-Id: I031e8b6cc4ab20fb094dbd46720829b78951e7f9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/146218
Commit-Queue: Erik Språng <sprang@webrtc.org>
Reviewed-by: Stefan Holmer <stefan@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28657}