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

Unified Diff: webrtc/media/engine/apm_helpers_unittest.cc

Issue 2961723004: Allow an external audio processing module to be used in WebRTC (Closed)
Patch Set: Moved creation of APMs from CreateVoiceEngines Created 3 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/media/base/mediaengine.h ('k') | webrtc/media/engine/fakewebrtcvoiceengine.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/engine/apm_helpers_unittest.cc
diff --git a/webrtc/media/engine/apm_helpers_unittest.cc b/webrtc/media/engine/apm_helpers_unittest.cc
index 6845d9e454bb28df14f4497025b22b86daea7c3c..a86990301c9623041e590a70434098098bfa8b4f 100644
--- a/webrtc/media/engine/apm_helpers_unittest.cc
+++ b/webrtc/media/engine/apm_helpers_unittest.cc
@@ -33,19 +33,15 @@ struct TestHelper {
// This replicates the conditions from voe_auto_test.
Config config;
config.Set<ExperimentalAgc>(new ExperimentalAgc(false));
+ apm_ = rtc::scoped_refptr<AudioProcessing>(AudioProcessing::Create(config));
EXPECT_EQ(0, voe_wrapper_.base()->Init(
- &mock_audio_device_,
- AudioProcessing::Create(config),
- MockAudioDecoderFactory::CreateEmptyFactory()));
+ &mock_audio_device_, apm_,
+ MockAudioDecoderFactory::CreateEmptyFactory()));
}
- AudioProcessing* apm() {
- return voe_wrapper_.base()->audio_processing();
- }
+ AudioProcessing* apm() { return apm_.get(); }
- const AudioProcessing* apm() const {
- return voe_wrapper_.base()->audio_processing();
- }
+ const AudioProcessing* apm() const { return apm_.get(); }
test::MockAudioDeviceModule* adm() {
return &mock_audio_device_;
@@ -77,6 +73,7 @@ struct TestHelper {
private:
testing::NiceMock<test::MockAudioDeviceModule> mock_audio_device_;
cricket::VoEWrapper voe_wrapper_;
+ rtc::scoped_refptr<AudioProcessing> apm_;
};
} // namespace
« no previous file with comments | « webrtc/media/base/mediaengine.h ('k') | webrtc/media/engine/fakewebrtcvoiceengine.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698