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

Side by Side Diff: webrtc/test/call_test.cc

Issue 2652803002: Refactor FakeAudioDevice to have separate methods for starting recording and playout. (Closed)
Patch Set: Moved initialization to ctor. Made StartRecording and StartPlayout to private. Fixed call_test Created 3 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2014 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 122 matching lines...) Expand 10 before | Expand all | Expand 10 after
133 if (num_audio_streams_ > 0) 133 if (num_audio_streams_ > 0)
134 DestroyVoiceEngines(); 134 DestroyVoiceEngines();
135 } 135 }
136 136
137 void CallTest::Start() { 137 void CallTest::Start() {
138 if (video_send_stream_) 138 if (video_send_stream_)
139 video_send_stream_->Start(); 139 video_send_stream_->Start();
140 for (VideoReceiveStream* video_recv_stream : video_receive_streams_) 140 for (VideoReceiveStream* video_recv_stream : video_receive_streams_)
141 video_recv_stream->Start(); 141 video_recv_stream->Start();
142 if (audio_send_stream_) { 142 if (audio_send_stream_) {
143 fake_send_audio_device_->Start();
144 audio_send_stream_->Start(); 143 audio_send_stream_->Start();
145 EXPECT_EQ(0, voe_send_.base->StartSend(voe_send_.channel_id));
146 } 144 }
147 for (AudioReceiveStream* audio_recv_stream : audio_receive_streams_) 145 for (AudioReceiveStream* audio_recv_stream : audio_receive_streams_)
148 audio_recv_stream->Start(); 146 audio_recv_stream->Start();
149 if (!audio_receive_streams_.empty()) {
150 fake_recv_audio_device_->Start();
151 EXPECT_EQ(0, voe_recv_.base->StartPlayout(voe_recv_.channel_id));
152 }
153 for (FlexfecReceiveStream* flexfec_recv_stream : flexfec_receive_streams_) 147 for (FlexfecReceiveStream* flexfec_recv_stream : flexfec_receive_streams_)
154 flexfec_recv_stream->Start(); 148 flexfec_recv_stream->Start();
155 if (frame_generator_capturer_.get() != NULL) 149 if (frame_generator_capturer_.get() != NULL)
156 frame_generator_capturer_->Start(); 150 frame_generator_capturer_->Start();
157 } 151 }
158 152
159 void CallTest::Stop() { 153 void CallTest::Stop() {
160 if (frame_generator_capturer_.get() != NULL) 154 if (frame_generator_capturer_.get() != NULL)
161 frame_generator_capturer_->Stop(); 155 frame_generator_capturer_->Stop();
162 for (FlexfecReceiveStream* flexfec_recv_stream : flexfec_receive_streams_) 156 for (FlexfecReceiveStream* flexfec_recv_stream : flexfec_receive_streams_)
163 flexfec_recv_stream->Stop(); 157 flexfec_recv_stream->Stop();
164 if (!audio_receive_streams_.empty()) {
165 fake_recv_audio_device_->Stop();
166 EXPECT_EQ(0, voe_recv_.base->StopPlayout(voe_recv_.channel_id));
167 }
168 for (AudioReceiveStream* audio_recv_stream : audio_receive_streams_) 158 for (AudioReceiveStream* audio_recv_stream : audio_receive_streams_)
169 audio_recv_stream->Stop(); 159 audio_recv_stream->Stop();
170 if (audio_send_stream_) { 160 if (audio_send_stream_) {
171 fake_send_audio_device_->Stop();
172 EXPECT_EQ(0, voe_send_.base->StopSend(voe_send_.channel_id));
173 audio_send_stream_->Stop(); 161 audio_send_stream_->Stop();
174 } 162 }
175 for (VideoReceiveStream* video_recv_stream : video_receive_streams_) 163 for (VideoReceiveStream* video_recv_stream : video_receive_streams_)
176 video_recv_stream->Stop(); 164 video_recv_stream->Stop();
177 if (video_send_stream_) 165 if (video_send_stream_)
178 video_send_stream_->Stop(); 166 video_send_stream_->Stop();
179 } 167 }
180 168
181 void CallTest::CreateCalls(const Call::Config& sender_config, 169 void CallTest::CreateCalls(const Call::Config& sender_config,
182 const Call::Config& receiver_config) { 170 const Call::Config& receiver_config) {
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
302 int width, 290 int width,
303 int height) { 291 int height) {
304 frame_generator_capturer_.reset( 292 frame_generator_capturer_.reset(
305 test::FrameGeneratorCapturer::Create(width, height, framerate, clock_)); 293 test::FrameGeneratorCapturer::Create(width, height, framerate, clock_));
306 video_send_stream_->SetSource( 294 video_send_stream_->SetSource(
307 frame_generator_capturer_.get(), 295 frame_generator_capturer_.get(),
308 VideoSendStream::DegradationPreference::kBalanced); 296 VideoSendStream::DegradationPreference::kBalanced);
309 } 297 }
310 298
311 void CallTest::CreateFakeAudioDevices() { 299 void CallTest::CreateFakeAudioDevices() {
312 fake_send_audio_device_.reset(new FakeAudioDevice( 300 fake_send_audio_device_.reset(new FakeAudioDevice(1.f, 48000, 256));
313 clock_, test::ResourcePath("voice_engine/audio_long16", "pcm"), 301 fake_recv_audio_device_.reset(new FakeAudioDevice(1.f, 48000, 256));
314 DriftingClock::kNoDrift));
315 fake_recv_audio_device_.reset(new FakeAudioDevice(
316 clock_, test::ResourcePath("voice_engine/audio_long16", "pcm"),
317 DriftingClock::kNoDrift));
318 } 302 }
319 303
320 void CallTest::CreateVideoStreams() { 304 void CallTest::CreateVideoStreams() {
321 RTC_DCHECK(video_send_stream_ == nullptr); 305 RTC_DCHECK(video_send_stream_ == nullptr);
322 RTC_DCHECK(video_receive_streams_.empty()); 306 RTC_DCHECK(video_receive_streams_.empty());
323 RTC_DCHECK(audio_send_stream_ == nullptr); 307 RTC_DCHECK(audio_send_stream_ == nullptr);
324 RTC_DCHECK(audio_receive_streams_.empty()); 308 RTC_DCHECK(audio_receive_streams_.empty());
325 309
326 video_send_stream_ = sender_call_->CreateVideoSendStream( 310 video_send_stream_ = sender_call_->CreateVideoSendStream(
327 video_send_config_.Copy(), video_encoder_config_.Copy()); 311 video_send_config_.Copy(), video_encoder_config_.Copy());
(...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
509 493
510 EndToEndTest::EndToEndTest(unsigned int timeout_ms) : BaseTest(timeout_ms) { 494 EndToEndTest::EndToEndTest(unsigned int timeout_ms) : BaseTest(timeout_ms) {
511 } 495 }
512 496
513 bool EndToEndTest::ShouldCreateReceivers() const { 497 bool EndToEndTest::ShouldCreateReceivers() const {
514 return true; 498 return true;
515 } 499 }
516 500
517 } // namespace test 501 } // namespace test
518 } // namespace webrtc 502 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698