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

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

Issue 2725823002: Move delay_based_bwe_ into CongestionController (Closed)
Patch Set: UT thread-checking. 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
« no previous file with comments | « webrtc/audio/audio_send_stream.cc ('k') | webrtc/modules/congestion_controller/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
144 EXPECT_CALL(*channel_proxy_, 144 EXPECT_CALL(*channel_proxy_,
145 SetSendAudioLevelIndicationStatus(true, kAudioLevelId)) 145 SetSendAudioLevelIndicationStatus(true, kAudioLevelId))
146 .Times(1); 146 .Times(1);
147 147
148 if (audio_bwe_enabled) { 148 if (audio_bwe_enabled) {
149 EXPECT_CALL(*channel_proxy_, 149 EXPECT_CALL(*channel_proxy_,
150 EnableSendTransportSequenceNumber(kTransportSequenceNumberId)) 150 EnableSendTransportSequenceNumber(kTransportSequenceNumberId))
151 .Times(1); 151 .Times(1);
152 EXPECT_CALL(*channel_proxy_, 152 EXPECT_CALL(*channel_proxy_,
153 RegisterSenderCongestionControlObjects( 153 RegisterSenderCongestionControlObjects(
154 congestion_controller_.pacer(), 154 congestion_controller_.pacer(), &congestion_controller_,
155 congestion_controller_.GetTransportFeedbackObserver(),
156 packet_router(), Ne(nullptr))) 155 packet_router(), Ne(nullptr)))
157 .Times(1); 156 .Times(1);
158 } else { 157 } else {
159 EXPECT_CALL(*channel_proxy_, 158 EXPECT_CALL(*channel_proxy_,
160 RegisterSenderCongestionControlObjects( 159 RegisterSenderCongestionControlObjects(
161 congestion_controller_.pacer(), 160 congestion_controller_.pacer(), &congestion_controller_,
162 congestion_controller_.GetTransportFeedbackObserver(),
163 packet_router(), Eq(nullptr))) 161 packet_router(), Eq(nullptr)))
164 .Times(1); 162 .Times(1);
165 } 163 }
166 EXPECT_CALL(*channel_proxy_, ResetCongestionControlObjects()).Times(1); 164 EXPECT_CALL(*channel_proxy_, ResetCongestionControlObjects()).Times(1);
167 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(nullptr)).Times(1); 165 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(nullptr)).Times(1);
168 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1); 166 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1);
169 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1); 167 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1);
170 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull())) 168 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull()))
171 .Times(1); // Destructor resets the event log 169 .Times(1); // Destructor resets the event log
172 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1); 170 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1);
(...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
467 internal::AudioSendStream send_stream( 465 internal::AudioSendStream send_stream(
468 helper.config(), helper.audio_state(), helper.worker_queue(), 466 helper.config(), helper.audio_state(), helper.worker_queue(),
469 helper.packet_router(), helper.congestion_controller(), 467 helper.packet_router(), helper.congestion_controller(),
470 helper.bitrate_allocator(), helper.event_log(), helper.rtcp_rtt_stats()); 468 helper.bitrate_allocator(), helper.event_log(), helper.rtcp_rtt_stats());
471 EXPECT_CALL(*helper.channel_proxy(), SetBitrate(_, 5000)); 469 EXPECT_CALL(*helper.channel_proxy(), SetBitrate(_, 5000));
472 send_stream.OnBitrateUpdated(50000, 0.0, 50, 5000); 470 send_stream.OnBitrateUpdated(50000, 0.0, 50, 5000);
473 } 471 }
474 472
475 } // namespace test 473 } // namespace test
476 } // namespace webrtc 474 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/audio/audio_send_stream.cc ('k') | webrtc/modules/congestion_controller/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698