Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(202)

Side by Side Diff: webrtc/call/congestion_controller.cc

Issue 1535963002: Wire-up BWE feedback for audio receive streams. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Comments addressed. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 packet_router_.get())), 162 packet_router_.get())),
163 process_thread_(process_thread), 163 process_thread_(process_thread),
164 call_stats_(call_stats), 164 call_stats_(call_stats),
165 pacer_thread_(ProcessThread::Create("PacerThread")), 165 pacer_thread_(ProcessThread::Create("PacerThread")),
166 // Constructed last as this object calls the provided callback on 166 // Constructed last as this object calls the provided callback on
167 // construction. 167 // construction.
168 bitrate_controller_( 168 bitrate_controller_(
169 BitrateController::CreateBitrateController(Clock::GetRealTimeClock(), 169 BitrateController::CreateBitrateController(Clock::GetRealTimeClock(),
170 bitrate_observer)), 170 bitrate_observer)),
171 min_bitrate_bps_(RemoteBitrateEstimator::kDefaultMinBitrateBps) { 171 min_bitrate_bps_(RemoteBitrateEstimator::kDefaultMinBitrateBps) {
172 RTC_DCHECK(bitrate_observer);
mflodman 2016/01/12 12:42:53 I don't this check makes sense here, since it's no
stefan-webrtc 2016/01/12 12:53:18 I'll remove it for now.
172 call_stats_->RegisterStatsObserver(remote_bitrate_estimator_.get()); 173 call_stats_->RegisterStatsObserver(remote_bitrate_estimator_.get());
173 174
174 pacer_thread_->RegisterModule(pacer_.get()); 175 pacer_thread_->RegisterModule(pacer_.get());
175 pacer_thread_->Start(); 176 pacer_thread_->Start();
176 177
177 process_thread->RegisterModule(remote_estimator_proxy_.get()); 178 process_thread->RegisterModule(remote_estimator_proxy_.get());
178 process_thread->RegisterModule(remote_bitrate_estimator_.get()); 179 process_thread->RegisterModule(remote_bitrate_estimator_.get());
179 process_thread->RegisterModule(bitrate_controller_.get()); 180 process_thread->RegisterModule(bitrate_controller_.get());
180 } 181 }
181 182
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
285 } 286 }
286 } 287 }
287 288
288 void CongestionController::OnSentPacket(const rtc::SentPacket& sent_packet) { 289 void CongestionController::OnSentPacket(const rtc::SentPacket& sent_packet) {
289 if (transport_feedback_adapter_) { 290 if (transport_feedback_adapter_) {
290 transport_feedback_adapter_->OnSentPacket(sent_packet.packet_id, 291 transport_feedback_adapter_->OnSentPacket(sent_packet.packet_id,
291 sent_packet.send_time_ms); 292 sent_packet.send_time_ms);
292 } 293 }
293 } 294 }
294 } // namespace webrtc 295 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698