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

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

Issue 2774623006: Let PacketRouter separate send and receive modules. (Closed)
Patch Set: Delete #if:ed out code. Created 3 years, 8 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 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 EXPECT_CALL(*channel_proxy_, RegisterSenderCongestionControlObjects( 149 EXPECT_CALL(*channel_proxy_, RegisterSenderCongestionControlObjects(
150 send_side_cc_.pacer(), &send_side_cc_, 150 send_side_cc_.pacer(), &send_side_cc_,
151 packet_router(), Ne(nullptr))) 151 packet_router(), Ne(nullptr)))
152 .Times(1); 152 .Times(1);
153 } else { 153 } else {
154 EXPECT_CALL(*channel_proxy_, RegisterSenderCongestionControlObjects( 154 EXPECT_CALL(*channel_proxy_, RegisterSenderCongestionControlObjects(
155 send_side_cc_.pacer(), &send_side_cc_, 155 send_side_cc_.pacer(), &send_side_cc_,
156 packet_router(), Eq(nullptr))) 156 packet_router(), Eq(nullptr)))
157 .Times(1); 157 .Times(1);
158 } 158 }
159 EXPECT_CALL(*channel_proxy_, ResetCongestionControlObjects()).Times(1); 159 EXPECT_CALL(*channel_proxy_, ResetSenderCongestionControlObjects())
160 .Times(1);
160 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(nullptr)).Times(1); 161 EXPECT_CALL(*channel_proxy_, RegisterExternalTransport(nullptr)).Times(1);
161 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1); 162 EXPECT_CALL(*channel_proxy_, DeRegisterExternalTransport()).Times(1);
162 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1); 163 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::NotNull())).Times(1);
163 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull())) 164 EXPECT_CALL(*channel_proxy_, SetRtcEventLog(testing::IsNull()))
164 .Times(1); // Destructor resets the event log 165 .Times(1); // Destructor resets the event log
165 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1); 166 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(&rtcp_rtt_stats_)).Times(1);
166 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(testing::IsNull())) 167 EXPECT_CALL(*channel_proxy_, SetRtcpRttStats(testing::IsNull()))
167 .Times(1); // Destructor resets the rtt stats. 168 .Times(1); // Destructor resets the rtt stats.
168 } 169 }
169 170
(...skipping 290 matching lines...) Expand 10 before | Expand all | Expand 10 after
460 internal::AudioSendStream send_stream( 461 internal::AudioSendStream send_stream(
461 helper.config(), helper.audio_state(), helper.worker_queue(), 462 helper.config(), helper.audio_state(), helper.worker_queue(),
462 helper.packet_router(), helper.send_side_cc(), helper.bitrate_allocator(), 463 helper.packet_router(), helper.send_side_cc(), helper.bitrate_allocator(),
463 helper.event_log(), helper.rtcp_rtt_stats()); 464 helper.event_log(), helper.rtcp_rtt_stats());
464 EXPECT_CALL(*helper.channel_proxy(), SetBitrate(_, 5000)); 465 EXPECT_CALL(*helper.channel_proxy(), SetBitrate(_, 5000));
465 send_stream.OnBitrateUpdated(50000, 0.0, 50, 5000); 466 send_stream.OnBitrateUpdated(50000, 0.0, 50, 5000);
466 } 467 }
467 468
468 } // namespace test 469 } // namespace test
469 } // namespace webrtc 470 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698