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

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

Issue 1628683002: Use separate rtp module lists for send and receive in PacketRouter. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Comments addressed. Created 4 years, 10 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_receive_stream.cc ('k') | webrtc/audio/audio_send_stream.cc » ('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 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 EXPECT_CALL(*channel_proxy_, SetLocalSSRC(kLocalSsrc)).Times(1); 87 EXPECT_CALL(*channel_proxy_, SetLocalSSRC(kLocalSsrc)).Times(1);
88 EXPECT_CALL(*channel_proxy_, 88 EXPECT_CALL(*channel_proxy_,
89 SetReceiveAbsoluteSenderTimeStatus(true, kAbsSendTimeId)) 89 SetReceiveAbsoluteSenderTimeStatus(true, kAbsSendTimeId))
90 .Times(1); 90 .Times(1);
91 EXPECT_CALL(*channel_proxy_, 91 EXPECT_CALL(*channel_proxy_,
92 SetReceiveAudioLevelIndicationStatus(true, kAudioLevelId)) 92 SetReceiveAudioLevelIndicationStatus(true, kAudioLevelId))
93 .Times(1); 93 .Times(1);
94 EXPECT_CALL(*channel_proxy_, EnableReceiveTransportSequenceNumber( 94 EXPECT_CALL(*channel_proxy_, EnableReceiveTransportSequenceNumber(
95 kTransportSequenceNumberId)) 95 kTransportSequenceNumberId))
96 .Times(1); 96 .Times(1);
97 EXPECT_CALL(*channel_proxy_, SetCongestionControlObjects( 97 EXPECT_CALL(*channel_proxy_,
98 nullptr, nullptr, &packet_router_)) 98 RegisterReceiverCongestionControlObjects(&packet_router_))
99 .Times(1); 99 .Times(1);
100 EXPECT_CALL(congestion_controller_, packet_router()) 100 EXPECT_CALL(congestion_controller_, packet_router())
101 .WillOnce(Return(&packet_router_)); 101 .WillOnce(Return(&packet_router_));
102 EXPECT_CALL(*channel_proxy_, 102 EXPECT_CALL(*channel_proxy_, ResetCongestionControlObjects())
103 SetCongestionControlObjects(nullptr, nullptr, nullptr))
104 .Times(1); 103 .Times(1);
105 return channel_proxy_; 104 return channel_proxy_;
106 })); 105 }));
107 stream_config_.voe_channel_id = kChannelId; 106 stream_config_.voe_channel_id = kChannelId;
108 stream_config_.rtp.local_ssrc = kLocalSsrc; 107 stream_config_.rtp.local_ssrc = kLocalSsrc;
109 stream_config_.rtp.remote_ssrc = kRemoteSsrc; 108 stream_config_.rtp.remote_ssrc = kRemoteSsrc;
110 stream_config_.rtp.extensions.push_back( 109 stream_config_.rtp.extensions.push_back(
111 RtpExtension(RtpExtension::kAbsSendTime, kAbsSendTimeId)); 110 RtpExtension(RtpExtension::kAbsSendTime, kAbsSendTimeId));
112 stream_config_.rtp.extensions.push_back( 111 stream_config_.rtp.extensions.push_back(
113 RtpExtension(RtpExtension::kAudioLevel, kAudioLevelId)); 112 RtpExtension(RtpExtension::kAudioLevel, kAudioLevelId));
(...skipping 209 matching lines...) Expand 10 before | Expand all | Expand 10 after
323 EXPECT_EQ(kAudioDecodeStats.calls_to_neteq, stats.decoding_calls_to_neteq); 322 EXPECT_EQ(kAudioDecodeStats.calls_to_neteq, stats.decoding_calls_to_neteq);
324 EXPECT_EQ(kAudioDecodeStats.decoded_normal, stats.decoding_normal); 323 EXPECT_EQ(kAudioDecodeStats.decoded_normal, stats.decoding_normal);
325 EXPECT_EQ(kAudioDecodeStats.decoded_plc, stats.decoding_plc); 324 EXPECT_EQ(kAudioDecodeStats.decoded_plc, stats.decoding_plc);
326 EXPECT_EQ(kAudioDecodeStats.decoded_cng, stats.decoding_cng); 325 EXPECT_EQ(kAudioDecodeStats.decoded_cng, stats.decoding_cng);
327 EXPECT_EQ(kAudioDecodeStats.decoded_plc_cng, stats.decoding_plc_cng); 326 EXPECT_EQ(kAudioDecodeStats.decoded_plc_cng, stats.decoding_plc_cng);
328 EXPECT_EQ(kCallStats.capture_start_ntp_time_ms_, 327 EXPECT_EQ(kCallStats.capture_start_ntp_time_ms_,
329 stats.capture_start_ntp_time_ms); 328 stats.capture_start_ntp_time_ms);
330 } 329 }
331 } // namespace test 330 } // namespace test
332 } // namespace webrtc 331 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/audio/audio_receive_stream.cc ('k') | webrtc/audio/audio_send_stream.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698