commit 04c94ad4916e121368ee20867eaa333e14222cd2
author Tommi <tommi@webrtc.org> Sat May 16 09:52:50 2020
committer Commit Bot <commit-bot@chromium.org> Sun May 17 17:17:50 2020
tree 5b6672c9b99995daf64f45ead89d82f0d2edbb58
parent ec3ba734e935eb58263ae21ac11a68265fa08473

Fix misleading SequenceChecker variable name.

I named the network sequence checker 'worker' some time ago.
It represents the network TQ in RtpTransportControllerSendInterface
though, so should rather be called 'network'.

Bug: none
Change-Id: If82d7528b8cfcc180e1515b2fad63ed01610ec32
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/175340
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31291}

call/call.cc

diff --git a/call/call.cc b/call/call.cc
index 4068db9..210f72d 100644
--- a/call/call.cc
+++ b/call/call.cc
@@ -275,7 +275,7 @@
   const std::unique_ptr<BitrateAllocator> bitrate_allocator_;
   Call::Config config_;
   SequenceChecker configuration_sequence_checker_;
-  SequenceChecker worker_sequence_checker_;
+  SequenceChecker network_sequence_checker_;
 
   NetworkState audio_network_state_;
   NetworkState video_network_state_;
@@ -364,7 +364,7 @@
   // OnNetworkChanged from multiple threads.
   rtc::CriticalSection bitrate_crit_;
   uint32_t min_allocated_send_bitrate_bps_
-      RTC_GUARDED_BY(&worker_sequence_checker_);
+      RTC_GUARDED_BY(&network_sequence_checker_);
   uint32_t configured_max_padding_bitrate_bps_ RTC_GUARDED_BY(&bitrate_crit_);
   AvgCounter estimated_send_bitrate_kbps_counter_
       RTC_GUARDED_BY(&bitrate_crit_);
@@ -473,7 +473,7 @@
       transport_send_(std::move(transport_send)) {
   RTC_DCHECK(config.event_log != nullptr);
   RTC_DCHECK(config.trials != nullptr);
-  worker_sequence_checker_.Detach();
+  network_sequence_checker_.Detach();
 
   call_stats_->RegisterStatsObserver(&receive_side_cc_);
 
@@ -1079,7 +1079,7 @@
 
 void Call::OnTargetTransferRate(TargetTransferRate msg) {
   RTC_DCHECK(network_queue()->IsCurrent());
-  RTC_DCHECK_RUN_ON(&worker_sequence_checker_);
+  RTC_DCHECK_RUN_ON(&network_sequence_checker_);
   {
     rtc::CritScope cs(&last_bandwidth_bps_crit_);
     last_bandwidth_bps_ = msg.target_rate.bps();
@@ -1120,7 +1120,7 @@
 
 void Call::OnAllocationLimitsChanged(BitrateAllocationLimits limits) {
   RTC_DCHECK(network_queue()->IsCurrent());
-  RTC_DCHECK_RUN_ON(&worker_sequence_checker_);
+  RTC_DCHECK_RUN_ON(&network_sequence_checker_);
 
   transport_send_ptr_->SetAllocatedSendBitrateLimits(limits);