commit f075917cb047df4867bc18e52d2c9a608118a62b
author Per Kjellander <perkj@webrtc.org> Fri Apr 09 11:41:53 2021
committer Commit Bot <commit-bot@chromium.org> Mon Apr 12 06:13:13 2021
tree fab580aa934673b968d4611a2f6a0c30fa3baf56
parent 1c73e03a37c20bcccba0e21561b69723b3a521da

Ensure TaskQueuePacedSender dont depend on PacketRouter
TaskQueuePacedSender only needs PacingController::PacketSender

Bug: None
Change-Id: I5f9aaa51f48efc099caaef474f14fd37334a52d3
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/214781
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Per Kjellander <perkj@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33680}

modules/pacing/task_queue_paced_sender.cc

diff --git a/modules/pacing/task_queue_paced_sender.cc b/modules/pacing/task_queue_paced_sender.cc
index 0944741..709718f 100644
--- a/modules/pacing/task_queue_paced_sender.cc
+++ b/modules/pacing/task_queue_paced_sender.cc
@@ -32,7 +32,7 @@
 
 TaskQueuePacedSender::TaskQueuePacedSender(
     Clock* clock,
-    PacketRouter* packet_router,
+    PacingController::PacketSender* packet_sender,
     RtcEventLog* event_log,
     const WebRtcKeyValueConfig* field_trials,
     TaskQueueFactory* task_queue_factory,
@@ -40,7 +40,7 @@
     : clock_(clock),
       hold_back_window_(hold_back_window),
       pacing_controller_(clock,
-                         packet_router,
+                         packet_sender,
                          event_log,
                          field_trials,
                          PacingController::ProcessMode::kDynamic),

modules/pacing/task_queue_paced_sender.h

diff --git a/modules/pacing/task_queue_paced_sender.h b/modules/pacing/task_queue_paced_sender.h
index a593069..e04fc19 100644
--- a/modules/pacing/task_queue_paced_sender.h
+++ b/modules/pacing/task_queue_paced_sender.h
@@ -27,7 +27,6 @@
 #include "api/units/timestamp.h"
 #include "modules/include/module.h"
 #include "modules/pacing/pacing_controller.h"
-#include "modules/pacing/packet_router.h"
 #include "modules/pacing/rtp_packet_pacer.h"
 #include "modules/rtp_rtcp/source/rtp_packet_to_send.h"
 #include "rtc_base/synchronization/mutex.h"
@@ -47,7 +46,7 @@
   // TODO(bugs.webrtc.org/10809): Remove default value for hold_back_window.
   TaskQueuePacedSender(
       Clock* clock,
-      PacketRouter* packet_router,
+      PacingController::PacketSender* packet_sender,
       RtcEventLog* event_log,
       const WebRtcKeyValueConfig* field_trials,
       TaskQueueFactory* task_queue_factory,
@@ -60,8 +59,8 @@
 
   // Methods implementing RtpPacketSender.
 
-  // Adds the packet to the queue and calls PacketRouter::SendPacket() when
-  // it's time to send.
+  // Adds the packet to the queue and calls
+  // PacingController::PacketSender::SendPacket() when it's time to send.
   void EnqueuePackets(
       std::vector<std::unique_ptr<RtpPacketToSend>> packets) override;