commit 0a6510ddf92c08b29a9762b3318a71536bfd85d5
author Sebastian Jansson <srte@webrtc.org> Fri Oct 04 07:31:08 2019
committer Commit Bot <commit-bot@chromium.org> Mon Oct 07 12:58:55 2019
tree 21d03e84fa1ec3c3f388fc4967cdf51f9c971744
parent 99a20962483581349f8cd1c37ef7b4cc79dd9ca5

Removes rtp_transport checks in AudioSendStream

There's already a DCHECK at construction time ensuring that it's set.

Bug: webrtC:9883
Change-Id: I9f41b77273bb859626546ab3534d483d9172ea5d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/155581
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Reviewed-by: Oskar Sundbom <ossu@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#29393}

audio/audio_send_stream.cc

diff --git a/audio/audio_send_stream.cc b/audio/audio_send_stream.cc
index 401e859..7f42d73 100644
--- a/audio/audio_send_stream.cc
+++ b/audio/audio_send_stream.cc
@@ -195,21 +195,16 @@
   ConfigureStream(config, true);
 
   pacer_thread_checker_.Detach();
-  if (rtp_transport_) {
-    // Signal congestion controller this object is ready for OnPacket*
-    // callbacks.
-    rtp_transport_->RegisterPacketFeedbackObserver(this);
-  }
+  // Signal congestion controller this object is ready for OnPacket* callbacks.
+  rtp_transport_->RegisterPacketFeedbackObserver(this);
 }
 
 AudioSendStream::~AudioSendStream() {
   RTC_DCHECK(worker_thread_checker_.IsCurrent());
   RTC_LOG(LS_INFO) << "~AudioSendStream: " << config_.rtp.ssrc;
   RTC_DCHECK(!sending_);
-  if (rtp_transport_) {
-    rtp_transport_->DeRegisterPacketFeedbackObserver(this);
-    channel_send_->ResetSenderCongestionControlObjects();
-  }
+  rtp_transport_->DeRegisterPacketFeedbackObserver(this);
+  channel_send_->ResetSenderCongestionControlObjects();
   // Blocking call to synchronize state with worker queue to ensure that there
   // are no pending tasks left that keeps references to audio.
   rtc::Event thread_sync_event;
@@ -323,19 +318,15 @@
       // Probing in application limited region is only used in combination with
       // send side congestion control, wich depends on feedback packets which
       // requires transport sequence numbers to be enabled.
-      if (rtp_transport_) {
-        // Optionally request ALR probing but do not override any existing
-        // request from other streams.
-        if (enable_audio_alr_probing_) {
-          rtp_transport_->EnablePeriodicAlrProbing(true);
-        }
-        bandwidth_observer = rtp_transport_->GetBandwidthObserver();
+      // Optionally request ALR probing but do not override any existing
+      // request from other streams.
+      if (enable_audio_alr_probing_) {
+        rtp_transport_->EnablePeriodicAlrProbing(true);
       }
+      bandwidth_observer = rtp_transport_->GetBandwidthObserver();
     }
-    if (rtp_transport_) {
-      channel_send_->RegisterSenderCongestionControlObjects(rtp_transport_,
-                                                            bandwidth_observer);
-    }
+    channel_send_->RegisterSenderCongestionControlObjects(rtp_transport_,
+                                                          bandwidth_observer);
   }
   config_cs_.Enter();
   // MID RTP header extension.