commit c96aa30c1397e1f5e9cda06330413e946b55ea13
author Erik Språng <sprang@webrtc.org> Thu Feb 25 16:05:21 2021
committer Commit Bot <commit-bot@chromium.org> Fri Feb 26 13:20:52 2021
tree a99034ca4c6fd1e469bbd4928dc8f3c43a71313d
parent 484acf27231d931dbc99aedce85bc27e06486b96

Minor refactoring of RtpVideoSender

Bug: None
Change-Id: I7cc16d4e435974090310bd3a7cea9eaf0586be8c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/208761
Reviewed-by: Niels Moller <nisse@webrtc.org>
Commit-Queue: Erik Språng <sprang@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33351}

call/rtp_video_sender.cc

diff --git a/call/rtp_video_sender.cc b/call/rtp_video_sender.cc
index 732d17a..1f17c7e 100644
--- a/call/rtp_video_sender.cc
+++ b/call/rtp_video_sender.cc
@@ -497,28 +497,27 @@
       active_ = true;
     }
 
-    const bool was_active = rtp_streams_[i].rtp_rtcp->SendingMedia();
+    RtpRtcpInterface& rtp_module = *rtp_streams_[i].rtp_rtcp;
+    const bool was_active = rtp_module.SendingMedia();
     const bool should_be_active = active_modules[i];
 
     // Sends a kRtcpByeCode when going from true to false.
-    rtp_streams_[i].rtp_rtcp->SetSendingStatus(active_modules[i]);
+    rtp_module.SetSendingStatus(active_modules[i]);
 
     if (was_active && !should_be_active) {
       // Disabling media, remove from packet router map to reduce size and
       // prevent any stray packets in the pacer from asynchronously arriving
       // to a disabled module.
-      transport_->packet_router()->RemoveSendRtpModule(
-          rtp_streams_[i].rtp_rtcp.get());
+      transport_->packet_router()->RemoveSendRtpModule(&rtp_module);
     }
 
     // If set to false this module won't send media.
-    rtp_streams_[i].rtp_rtcp->SetSendingMediaStatus(active_modules[i]);
+    rtp_module.SetSendingMediaStatus(active_modules[i]);
 
     if (!was_active && should_be_active) {
       // Turning on media, register with packet router.
-      transport_->packet_router()->AddSendRtpModule(
-          rtp_streams_[i].rtp_rtcp.get(),
-          /*remb_candidate=*/true);
+      transport_->packet_router()->AddSendRtpModule(&rtp_module,
+                                                    /*remb_candidate=*/true);
     }
   }
 }