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

Side by Side Diff: webrtc/call/call_unittest.cc

Issue 2469743002: Passed AudioMixer to AudioState::Config. (Closed)
Patch Set: 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
11 #include <list> 11 #include <list>
12 #include <memory> 12 #include <memory>
13 13
14 #include "webrtc/api/call/audio_state.h" 14 #include "webrtc/api/call/audio_state.h"
15 #include "webrtc/call.h" 15 #include "webrtc/call.h"
16 #include "webrtc/logging/rtc_event_log/rtc_event_log.h" 16 #include "webrtc/logging/rtc_event_log/rtc_event_log.h"
17 #include "webrtc/modules/audio_mixer/audio_mixer_impl.h"
17 #include "webrtc/modules/audio_coding/codecs/mock/mock_audio_decoder_factory.h" 18 #include "webrtc/modules/audio_coding/codecs/mock/mock_audio_decoder_factory.h"
18 #include "webrtc/test/gtest.h" 19 #include "webrtc/test/gtest.h"
19 #include "webrtc/test/mock_voice_engine.h" 20 #include "webrtc/test/mock_voice_engine.h"
20 21
21 namespace { 22 namespace {
22 23
23 struct CallHelper { 24 struct CallHelper {
24 explicit CallHelper( 25 explicit CallHelper(
25 rtc::scoped_refptr<webrtc::AudioDecoderFactory> decoder_factory = nullptr) 26 rtc::scoped_refptr<webrtc::AudioDecoderFactory> decoder_factory = nullptr)
26 : voice_engine_(decoder_factory) { 27 : voice_engine_(decoder_factory) {
27 webrtc::AudioState::Config audio_state_config; 28 webrtc::AudioState::Config audio_state_config;
28 audio_state_config.voice_engine = &voice_engine_; 29 audio_state_config.voice_engine = &voice_engine_;
29 EXPECT_CALL(voice_engine_, audio_device_module()); 30 EXPECT_CALL(voice_engine_, audio_device_module());
30 EXPECT_CALL(voice_engine_, audio_processing()); 31 EXPECT_CALL(voice_engine_, audio_processing());
31 EXPECT_CALL(voice_engine_, audio_transport()); 32 EXPECT_CALL(voice_engine_, audio_transport());
32 webrtc::Call::Config config(&event_log_); 33 webrtc::Call::Config config(&event_log_);
34 audio_state_config.audio_mixer = webrtc::AudioMixerImpl::Create();
the sun 2016/11/14 20:14:22 Move up (line 29)
33 config.audio_state = webrtc::AudioState::Create(audio_state_config); 35 config.audio_state = webrtc::AudioState::Create(audio_state_config);
34 call_.reset(webrtc::Call::Create(config)); 36 call_.reset(webrtc::Call::Create(config));
35 } 37 }
36 38
37 webrtc::Call* operator->() { return call_.get(); } 39 webrtc::Call* operator->() { return call_.get(); }
38 40
39 private: 41 private:
40 testing::NiceMock<webrtc::test::MockVoiceEngine> voice_engine_; 42 testing::NiceMock<webrtc::test::MockVoiceEngine> voice_engine_;
41 webrtc::RtcEventLogNullImpl event_log_; 43 webrtc::RtcEventLogNullImpl event_log_;
42 std::unique_ptr<webrtc::Call> call_; 44 std::unique_ptr<webrtc::Call> call_;
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
185 stream = call->CreateFlexfecReceiveStream(config); 187 stream = call->CreateFlexfecReceiveStream(config);
186 EXPECT_NE(stream, nullptr); 188 EXPECT_NE(stream, nullptr);
187 streams.push_back(stream); 189 streams.push_back(stream);
188 190
189 for (auto s : streams) { 191 for (auto s : streams) {
190 call->DestroyFlexfecReceiveStream(s); 192 call->DestroyFlexfecReceiveStream(s);
191 } 193 }
192 } 194 }
193 195
194 } // namespace webrtc 196 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698