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

Side by Side Diff: webrtc/audio/audio_send_stream_unittest.cc

Issue 2725823002: Move delay_based_bwe_ into CongestionController (Closed)
Patch Set: More refactoring + UTs moved. Created 3 years, 9 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) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 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 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 EXPECT_CALL(*channel_proxy_, 139 EXPECT_CALL(*channel_proxy_,
140 SetSendAudioLevelIndicationStatus(true, kAudioLevelId)) 140 SetSendAudioLevelIndicationStatus(true, kAudioLevelId))
141 .Times(1); 141 .Times(1);
142 142
143 if (audio_bwe_enabled) { 143 if (audio_bwe_enabled) {
144 EXPECT_CALL(*channel_proxy_, 144 EXPECT_CALL(*channel_proxy_,
145 EnableSendTransportSequenceNumber(kTransportSequenceNumberId)) 145 EnableSendTransportSequenceNumber(kTransportSequenceNumberId))
146 .Times(1); 146 .Times(1);
147 EXPECT_CALL(*channel_proxy_, 147 EXPECT_CALL(*channel_proxy_,
148 RegisterSenderCongestionControlObjects( 148 RegisterSenderCongestionControlObjects(
149 congestion_controller_.pacer(), 149 congestion_controller_.pacer(), congestion_controller_,
150 congestion_controller_.GetTransportFeedbackObserver(),
151 packet_router(), Ne(nullptr))) 150 packet_router(), Ne(nullptr)))
152 .Times(1); 151 .Times(1);
153 } else { 152 } else {
154 EXPECT_CALL(*channel_proxy_, 153 EXPECT_CALL(*channel_proxy_,
155 RegisterSenderCongestionControlObjects( 154 RegisterSenderCongestionControlObjects(
156 congestion_controller_.pacer(), 155 congestion_controller_.pacer(), congestion_controller_,
157 congestion_controller_.GetTransportFeedbackObserver(),
158 packet_router(), Eq(nullptr))) 156 packet_router(), Eq(nullptr)))
159 .Times(1); 157 .Times(1);
160 } 158 }
161 EXPECT_CALL(*channel_proxy_, ResetCongestionControlObjects()).Times(1); 159 EXPECT_CALL(*channel_proxy_, ResetCongestionControlObjects()).Times(1);
162 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(nullptr)).Times(1); 160 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(nullptr)).Times(1);
163 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1); 161 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1);
164 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1); 162 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1);
165 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull())) 163 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull()))
166 .Times(1); // Destructor resets the event log 164 .Times(1); // Destructor resets the event log
167 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1); 165 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1);
(...skipping 291 matching lines...) Expand 10 before | Expand all | Expand 10 after
459 internal::AudioSendStream send_stream( 457 internal::AudioSendStream send_stream(
460 helper.config(), helper.audio_state(), helper.worker_queue(), 458 helper.config(), helper.audio_state(), helper.worker_queue(),
461 helper.packet_router(), helper.congestion_controller(), 459 helper.packet_router(), helper.congestion_controller(),
462 helper.bitrate_allocator(), helper.event_log(), helper.rtcp_rtt_stats()); 460 helper.bitrate_allocator(), helper.event_log(), helper.rtcp_rtt_stats());
463 EXPECT_CALL(*helper.channel_proxy(), SetBitrate(_, 5000)); 461 EXPECT_CALL(*helper.channel_proxy(), SetBitrate(_, 5000));
464 send_stream.OnBitrateUpdated(50000, 0.0, 50, 5000); 462 send_stream.OnBitrateUpdated(50000, 0.0, 50, 5000);
465 } 463 }
466 464
467 } // namespace test 465 } // namespace test
468 } // namespace webrtc 466 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698