Add RTP sequence number to TransportFeedbackObserver::AddPacket()
With this change, both the normal RTP and the transport-wide sequence
numbers are propagated with with AddPacket() call via a new
RtpPacketSendInfo struct, replacing the previous set of parameters.
The intent with this is that SendTimeHistory can hold a mapping from
transport-wide to rtp sequence numbers, and then via callbacks let the
RTP modules know when packets have been received by the remote end.
Bug: webrtc:8975
Change-Id: I6a24fc6282cbb041393752d39593c2867b242192
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/133021
Commit-Queue: Erik Språng <sprang@webrtc.org>
Reviewed-by: Stefan Holmer <stefan@webrtc.org>
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27708}diff --git a/call/rtp_transport_controller_send.cc b/call/rtp_transport_controller_send.cc
index 203d41e..7c8c4c3 100644
--- a/call/rtp_transport_controller_send.cc
+++ b/call/rtp_transport_controller_send.cc
@@ -408,15 +408,12 @@
});
}
-void RtpTransportControllerSend::AddPacket(uint32_t ssrc,
- uint16_t sequence_number,
- size_t length,
- const PacedPacketInfo& pacing_info) {
- if (send_side_bwe_with_overhead_) {
- length += transport_overhead_bytes_per_packet_;
- }
+void RtpTransportControllerSend::OnAddPacket(
+ const RtpPacketSendInfo& packet_info) {
transport_feedback_adapter_.AddPacket(
- ssrc, sequence_number, length, pacing_info,
+ packet_info,
+ send_side_bwe_with_overhead_ ? transport_overhead_bytes_per_packet_.load()
+ : 0,
Timestamp::ms(clock_->TimeInMilliseconds()));
}
