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

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

Issue 2454373002: Added an empty AudioTransportProxy to AudioState. (Closed)
Patch Set: Added unit test for recorded data path. Created 4 years, 1 month 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 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 congestion_controller_(&simulated_clock_, 71 congestion_controller_(&simulated_clock_,
72 &bitrate_observer_, 72 &bitrate_observer_,
73 &remote_bitrate_observer_, 73 &remote_bitrate_observer_,
74 &event_log_) { 74 &event_log_) {
75 using testing::Invoke; 75 using testing::Invoke;
76 76
77 EXPECT_CALL(voice_engine_, 77 EXPECT_CALL(voice_engine_,
78 RegisterVoiceEngineObserver(_)).WillOnce(Return(0)); 78 RegisterVoiceEngineObserver(_)).WillOnce(Return(0));
79 EXPECT_CALL(voice_engine_, 79 EXPECT_CALL(voice_engine_,
80 DeRegisterVoiceEngineObserver()).WillOnce(Return(0)); 80 DeRegisterVoiceEngineObserver()).WillOnce(Return(0));
81 EXPECT_CALL(voice_engine_, audio_processing());
the sun 2016/11/14 13:50:08 Q: Is .Times(1) the default?
aleloi 2016/11/14 14:24:42 Yes, the default is 1 when there is no .WillOnce o
82 EXPECT_CALL(voice_engine_, audio_device_module());
83 EXPECT_CALL(voice_engine_, audio_transport());
84
81 AudioState::Config config; 85 AudioState::Config config;
82 config.voice_engine = &voice_engine_; 86 config.voice_engine = &voice_engine_;
83 audio_state_ = AudioState::Create(config); 87 audio_state_ = AudioState::Create(config);
84 88
85 EXPECT_CALL(voice_engine_, ChannelProxyFactory(kChannelId)) 89 EXPECT_CALL(voice_engine_, ChannelProxyFactory(kChannelId))
86 .WillOnce(Invoke([this](int channel_id) { 90 .WillOnce(Invoke([this](int channel_id) {
87 EXPECT_FALSE(channel_proxy_); 91 EXPECT_FALSE(channel_proxy_);
88 channel_proxy_ = new testing::StrictMock<MockVoEChannelProxy>(); 92 channel_proxy_ = new testing::StrictMock<MockVoEChannelProxy>();
89 EXPECT_CALL(*channel_proxy_, SetLocalSSRC(kLocalSsrc)).Times(1); 93 EXPECT_CALL(*channel_proxy_, SetLocalSSRC(kLocalSsrc)).Times(1);
90 EXPECT_CALL(*channel_proxy_, SetNACKStatus(true, 15)).Times(1); 94 EXPECT_CALL(*channel_proxy_, SetNACKStatus(true, 15)).Times(1);
(...skipping 265 matching lines...) Expand 10 before | Expand all | Expand 10 after
356 ConfigHelper helper; 360 ConfigHelper helper;
357 internal::AudioReceiveStream recv_stream( 361 internal::AudioReceiveStream recv_stream(
358 helper.congestion_controller(), helper.config(), helper.audio_state(), 362 helper.congestion_controller(), helper.config(), helper.audio_state(),
359 helper.event_log()); 363 helper.event_log());
360 EXPECT_CALL(*helper.channel_proxy(), 364 EXPECT_CALL(*helper.channel_proxy(),
361 SetChannelOutputVolumeScaling(FloatEq(0.765f))); 365 SetChannelOutputVolumeScaling(FloatEq(0.765f)));
362 recv_stream.SetGain(0.765f); 366 recv_stream.SetGain(0.765f);
363 } 367 }
364 } // namespace test 368 } // namespace test
365 } // namespace webrtc 369 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698