blob: afc3ae56b199913a98d5a9301cb9a019fd506bdd [file] [log] [blame]
andrew@webrtc.org60730cf2014-01-07 17:45:09 +00001/*
2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
3 *
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
6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree.
9 */
10
Mirko Bonadei92ea95e2017-09-15 06:47:31 +020011#include "modules/audio_processing/audio_processing_impl.h"
andrew@webrtc.org60730cf2014-01-07 17:45:09 +000012
Per Åhgren2507f8c2020-03-19 12:33:29 +010013#include <array>
Alessio Bazzicae4498052018-12-17 09:44:06 +010014#include <memory>
15
Mirko Bonadeid9708072019-01-25 20:26:48 +010016#include "api/scoped_refptr.h"
Alessio Bazzicad2b97402018-08-09 14:23:11 +020017#include "modules/audio_processing/include/audio_processing.h"
Alessio Bazzicae4498052018-12-17 09:44:06 +010018#include "modules/audio_processing/test/echo_control_mock.h"
Mirko Bonadei92ea95e2017-09-15 06:47:31 +020019#include "modules/audio_processing/test/test_utils.h"
Alessio Bazzicae4498052018-12-17 09:44:06 +010020#include "rtc_base/checks.h"
Steve Anton10542f22019-01-11 09:11:00 -080021#include "rtc_base/ref_counted_object.h"
Mirko Bonadei92ea95e2017-09-15 06:47:31 +020022#include "test/gmock.h"
23#include "test/gtest.h"
andrew@webrtc.org60730cf2014-01-07 17:45:09 +000024
andrew@webrtc.org60730cf2014-01-07 17:45:09 +000025namespace webrtc {
peaha9cc40b2017-06-29 08:32:09 -070026namespace {
andrew@webrtc.org60730cf2014-01-07 17:45:09 +000027
Alessio Bazzicae4498052018-12-17 09:44:06 +010028using ::testing::Invoke;
29using ::testing::NotNull;
30
andrew@webrtc.org60730cf2014-01-07 17:45:09 +000031class MockInitialize : public AudioProcessingImpl {
32 public:
peah88ac8532016-09-12 16:47:25 -070033 explicit MockInitialize(const webrtc::Config& config)
34 : AudioProcessingImpl(config) {}
andrew@webrtc.org60730cf2014-01-07 17:45:09 +000035
andrew@webrtc.orge84978f2014-01-25 02:09:06 +000036 MOCK_METHOD0(InitializeLocked, int());
danilchap56359be2017-09-07 07:53:45 -070037 int RealInitializeLocked() RTC_NO_THREAD_SAFETY_ANALYSIS {
pbos@webrtc.org788acd12014-12-15 09:41:24 +000038 return AudioProcessingImpl::InitializeLocked();
39 }
peaha9cc40b2017-06-29 08:32:09 -070040
Niels Möller6f72f562017-10-19 13:15:17 +020041 MOCK_CONST_METHOD0(AddRef, void());
42 MOCK_CONST_METHOD0(Release, rtc::RefCountReleaseStatus());
andrew@webrtc.org60730cf2014-01-07 17:45:09 +000043};
44
Alessio Bazzicae4498052018-12-17 09:44:06 +010045// Creates MockEchoControl instances and provides a raw pointer access to
46// the next created one. The raw pointer is meant to be used with gmock.
47// Returning a pointer of the next created MockEchoControl instance is necessary
48// for the following reasons: (i) gmock expectations must be set before any call
49// occurs, (ii) APM is initialized the first time that
50// AudioProcessingImpl::ProcessStream() is called and the initialization leads
51// to the creation of a new EchoControl object.
52class MockEchoControlFactory : public EchoControlFactory {
53 public:
Mirko Bonadei317a1f02019-09-17 17:06:18 +020054 MockEchoControlFactory() : next_mock_(std::make_unique<MockEchoControl>()) {}
Alessio Bazzicae4498052018-12-17 09:44:06 +010055 // Returns a pointer to the next MockEchoControl that this factory creates.
56 MockEchoControl* GetNext() const { return next_mock_.get(); }
Per Åhgren4e5c7092019-11-01 20:44:11 +010057 std::unique_ptr<EchoControl> Create(int sample_rate_hz,
58 int num_render_channels,
59 int num_capture_channels) override {
Alessio Bazzicae4498052018-12-17 09:44:06 +010060 std::unique_ptr<EchoControl> mock = std::move(next_mock_);
Mirko Bonadei317a1f02019-09-17 17:06:18 +020061 next_mock_ = std::make_unique<MockEchoControl>();
Alessio Bazzicae4498052018-12-17 09:44:06 +010062 return mock;
63 }
64
65 private:
66 std::unique_ptr<MockEchoControl> next_mock_;
67};
68
Alessio Bazzicad2b97402018-08-09 14:23:11 +020069// Mocks EchoDetector and records the first samples of the last analyzed render
70// stream frame. Used to check what data is read by an EchoDetector
71// implementation injected into an APM.
72class TestEchoDetector : public EchoDetector {
73 public:
74 TestEchoDetector()
75 : analyze_render_audio_called_(false),
76 last_render_audio_first_sample_(0.f) {}
77 ~TestEchoDetector() override = default;
78 void AnalyzeRenderAudio(rtc::ArrayView<const float> render_audio) override {
79 last_render_audio_first_sample_ = render_audio[0];
80 analyze_render_audio_called_ = true;
81 }
82 void AnalyzeCaptureAudio(rtc::ArrayView<const float> capture_audio) override {
83 }
84 void Initialize(int capture_sample_rate_hz,
85 int num_capture_channels,
86 int render_sample_rate_hz,
87 int num_render_channels) override {}
88 EchoDetector::Metrics GetMetrics() const override { return {}; }
89 // Returns true if AnalyzeRenderAudio() has been called at least once.
90 bool analyze_render_audio_called() const {
91 return analyze_render_audio_called_;
92 }
93 // Returns the first sample of the last analyzed render frame.
94 float last_render_audio_first_sample() const {
95 return last_render_audio_first_sample_;
96 }
97
98 private:
99 bool analyze_render_audio_called_;
100 float last_render_audio_first_sample_;
101};
102
103// Mocks CustomProcessing and applies ProcessSample() to all the samples.
104// Meant to be injected into an APM to modify samples in a known and detectable
105// way.
106class TestRenderPreProcessor : public CustomProcessing {
107 public:
108 TestRenderPreProcessor() = default;
109 ~TestRenderPreProcessor() = default;
110 void Initialize(int sample_rate_hz, int num_channels) override {}
111 void Process(AudioBuffer* audio) override {
112 for (size_t k = 0; k < audio->num_channels(); ++k) {
Per Åhgrend47941e2019-08-22 11:51:13 +0200113 rtc::ArrayView<float> channel_view(audio->channels()[k],
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200114 audio->num_frames());
115 std::transform(channel_view.begin(), channel_view.end(),
116 channel_view.begin(), ProcessSample);
117 }
Mirko Bonadeic4dd7302019-02-25 09:12:02 +0100118 }
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200119 std::string ToString() const override { return "TestRenderPreProcessor"; }
120 void SetRuntimeSetting(AudioProcessing::RuntimeSetting setting) override {}
121 // Modifies a sample. This member is used in Process() to modify a frame and
122 // it is publicly visible to enable tests.
123 static constexpr float ProcessSample(float x) { return 2.f * x; }
124};
125
peaha9cc40b2017-06-29 08:32:09 -0700126} // namespace
127
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000128TEST(AudioProcessingImplTest, AudioParameterChangeTriggersInit) {
Per Åhgren2507f8c2020-03-19 12:33:29 +0100129 webrtc::Config webrtc_config;
130 MockInitialize mock(webrtc_config);
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000131 ON_CALL(mock, InitializeLocked())
132 .WillByDefault(Invoke(&mock, &MockInitialize::RealInitializeLocked));
133
134 EXPECT_CALL(mock, InitializeLocked()).Times(1);
135 mock.Initialize();
136
Per Åhgren2507f8c2020-03-19 12:33:29 +0100137 constexpr size_t kMaxSampleRateHz = 32000;
138 constexpr size_t kMaxNumChannels = 2;
139 std::array<int16_t, kMaxNumChannels * kMaxSampleRateHz / 100> frame;
140 frame.fill(0);
141 StreamConfig config(16000, 1, /*has_keyboard=*/false);
peah2ace3f92016-09-10 04:42:27 -0700142 // Call with the default parameters; there should be an init.
Per Åhgren4bdced52017-06-27 16:00:38 +0200143 EXPECT_CALL(mock, InitializeLocked()).Times(0);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100144 EXPECT_NOERR(mock.ProcessStream(frame.data(), config, config, frame.data(),
145 /*vad_result=*/nullptr));
146 EXPECT_NOERR(
147 mock.ProcessReverseStream(frame.data(), config, config, frame.data()));
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000148
149 // New sample rate. (Only impacts ProcessStream).
Per Åhgren2507f8c2020-03-19 12:33:29 +0100150 config = StreamConfig(32000, 1, /*has_keyboard=*/false);
Yves Gerey665174f2018-06-19 15:03:05 +0200151 EXPECT_CALL(mock, InitializeLocked()).Times(1);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100152 EXPECT_NOERR(mock.ProcessStream(frame.data(), config, config, frame.data(),
153 /*vad_result=*/nullptr));
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000154
155 // New number of channels.
peah2ace3f92016-09-10 04:42:27 -0700156 // TODO(peah): Investigate why this causes 2 inits.
Per Åhgren2507f8c2020-03-19 12:33:29 +0100157 config = StreamConfig(32000, 2, /*has_keyboard=*/false);
Yves Gerey665174f2018-06-19 15:03:05 +0200158 EXPECT_CALL(mock, InitializeLocked()).Times(2);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100159 EXPECT_NOERR(mock.ProcessStream(frame.data(), config, config, frame.data(),
160 /*vad_result=*/nullptr));
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000161 // ProcessStream sets num_channels_ == num_output_channels.
Per Åhgren2507f8c2020-03-19 12:33:29 +0100162 EXPECT_NOERR(
163 mock.ProcessReverseStream(frame.data(), config, config, frame.data()));
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000164
aluebsb0319552016-03-17 20:39:53 -0700165 // A new sample rate passed to ProcessReverseStream should cause an init.
Per Åhgren2507f8c2020-03-19 12:33:29 +0100166 config = StreamConfig(16000, 2, /*has_keyboard=*/false);
Alex Luebs5b830fe2016-03-08 17:52:52 +0100167 EXPECT_CALL(mock, InitializeLocked()).Times(1);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100168 EXPECT_NOERR(
169 mock.ProcessReverseStream(frame.data(), config, config, frame.data()));
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000170}
171
Alessio Bazzicac054e782018-04-16 12:10:09 +0200172TEST(AudioProcessingImplTest, UpdateCapturePreGainRuntimeSetting) {
Alex Loikob5c9a792018-04-16 16:31:22 +0200173 std::unique_ptr<AudioProcessing> apm(AudioProcessingBuilder().Create());
174 webrtc::AudioProcessing::Config apm_config;
175 apm_config.pre_amplifier.enabled = true;
176 apm_config.pre_amplifier.fixed_gain_factor = 1.f;
177 apm->ApplyConfig(apm_config);
178
Per Åhgren2507f8c2020-03-19 12:33:29 +0100179 constexpr int kSampleRateHz = 48000;
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200180 constexpr int16_t kAudioLevel = 10000;
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200181 constexpr size_t kNumChannels = 2;
Alex Loikob5c9a792018-04-16 16:31:22 +0200182
Per Åhgren2507f8c2020-03-19 12:33:29 +0100183 std::array<int16_t, kNumChannels * kSampleRateHz / 100> frame;
184 StreamConfig config(kSampleRateHz, kNumChannels, /*has_keyboard=*/false);
185 frame.fill(kAudioLevel);
186 apm->ProcessStream(frame.data(), config, config, frame.data(),
187 /*vad_result=*/nullptr);
188 EXPECT_EQ(frame[100], kAudioLevel)
Alex Loikob5c9a792018-04-16 16:31:22 +0200189 << "With factor 1, frame shouldn't be modified.";
190
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200191 constexpr float kGainFactor = 2.f;
Alex Loikob5c9a792018-04-16 16:31:22 +0200192 apm->SetRuntimeSetting(
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200193 AudioProcessing::RuntimeSetting::CreateCapturePreGain(kGainFactor));
Alex Loikob5c9a792018-04-16 16:31:22 +0200194
195 // Process for two frames to have time to ramp up gain.
196 for (int i = 0; i < 2; ++i) {
Per Åhgren2507f8c2020-03-19 12:33:29 +0100197 frame.fill(kAudioLevel);
198 apm->ProcessStream(frame.data(), config, config, frame.data(),
199 /*vad_result=*/nullptr);
Alex Loikob5c9a792018-04-16 16:31:22 +0200200 }
Per Åhgren2507f8c2020-03-19 12:33:29 +0100201 EXPECT_EQ(frame[100], kGainFactor * kAudioLevel)
Alex Loikob5c9a792018-04-16 16:31:22 +0200202 << "Frame should be amplified.";
Alessio Bazzicac054e782018-04-16 12:10:09 +0200203}
204
Alessio Bazzicae4498052018-12-17 09:44:06 +0100205TEST(AudioProcessingImplTest,
206 EchoControllerObservesPreAmplifierEchoPathGainChange) {
207 // Tests that the echo controller observes an echo path gain change when the
208 // pre-amplifier submodule changes the gain.
Mirko Bonadei317a1f02019-09-17 17:06:18 +0200209 auto echo_control_factory = std::make_unique<MockEchoControlFactory>();
Alessio Bazzicae4498052018-12-17 09:44:06 +0100210 const auto* echo_control_factory_ptr = echo_control_factory.get();
211
212 std::unique_ptr<AudioProcessing> apm(
213 AudioProcessingBuilder()
214 .SetEchoControlFactory(std::move(echo_control_factory))
215 .Create());
Sam Zackrisson41478c72019-10-15 10:10:26 +0200216 // Disable AGC.
Alessio Bazzicae4498052018-12-17 09:44:06 +0100217 webrtc::AudioProcessing::Config apm_config;
Sam Zackrisson41478c72019-10-15 10:10:26 +0200218 apm_config.gain_controller1.enabled = false;
Alessio Bazzicae4498052018-12-17 09:44:06 +0100219 apm_config.gain_controller2.enabled = false;
220 apm_config.pre_amplifier.enabled = true;
221 apm_config.pre_amplifier.fixed_gain_factor = 1.f;
222 apm->ApplyConfig(apm_config);
223
Alessio Bazzicae4498052018-12-17 09:44:06 +0100224 constexpr int16_t kAudioLevel = 10000;
225 constexpr size_t kSampleRateHz = 48000;
226 constexpr size_t kNumChannels = 2;
Per Åhgren2507f8c2020-03-19 12:33:29 +0100227 std::array<int16_t, kNumChannels * kSampleRateHz / 100> frame;
228 StreamConfig config(kSampleRateHz, kNumChannels, /*has_keyboard=*/false);
229 frame.fill(kAudioLevel);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100230
231 MockEchoControl* echo_control_mock = echo_control_factory_ptr->GetNext();
232
Per Åhgren0aefbf02019-08-23 21:29:17 +0200233 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Fredrik Hernqvistbf47f342019-05-09 10:50:31 +0200234 EXPECT_CALL(*echo_control_mock,
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100235 ProcessCapture(NotNull(), testing::_, /*echo_path_change=*/false))
Fredrik Hernqvistbf47f342019-05-09 10:50:31 +0200236 .Times(1);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100237 apm->ProcessStream(frame.data(), config, config, frame.data(),
238 /*vad_result=*/nullptr);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100239
Per Åhgren0aefbf02019-08-23 21:29:17 +0200240 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Fredrik Hernqvistbf47f342019-05-09 10:50:31 +0200241 EXPECT_CALL(*echo_control_mock,
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100242 ProcessCapture(NotNull(), testing::_, /*echo_path_change=*/true))
Fredrik Hernqvistbf47f342019-05-09 10:50:31 +0200243 .Times(1);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100244 apm->SetRuntimeSetting(
245 AudioProcessing::RuntimeSetting::CreateCapturePreGain(2.f));
Per Åhgren2507f8c2020-03-19 12:33:29 +0100246 apm->ProcessStream(frame.data(), config, config, frame.data(),
247 /*vad_result=*/nullptr);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100248}
249
250TEST(AudioProcessingImplTest,
251 EchoControllerObservesAnalogAgc1EchoPathGainChange) {
252 // Tests that the echo controller observes an echo path gain change when the
253 // AGC1 analog adaptive submodule changes the analog gain.
Mirko Bonadei317a1f02019-09-17 17:06:18 +0200254 auto echo_control_factory = std::make_unique<MockEchoControlFactory>();
Alessio Bazzicae4498052018-12-17 09:44:06 +0100255 const auto* echo_control_factory_ptr = echo_control_factory.get();
256
257 std::unique_ptr<AudioProcessing> apm(
258 AudioProcessingBuilder()
259 .SetEchoControlFactory(std::move(echo_control_factory))
260 .Create());
Alessio Bazzicae4498052018-12-17 09:44:06 +0100261 webrtc::AudioProcessing::Config apm_config;
Sam Zackrisson41478c72019-10-15 10:10:26 +0200262 // Enable AGC1.
263 apm_config.gain_controller1.enabled = true;
264 apm_config.gain_controller1.mode =
265 AudioProcessing::Config::GainController1::kAdaptiveAnalog;
Alessio Bazzicae4498052018-12-17 09:44:06 +0100266 apm_config.gain_controller2.enabled = false;
267 apm_config.pre_amplifier.enabled = false;
268 apm->ApplyConfig(apm_config);
269
Alessio Bazzicae4498052018-12-17 09:44:06 +0100270 constexpr int16_t kAudioLevel = 1000;
271 constexpr size_t kSampleRateHz = 48000;
272 constexpr size_t kNumChannels = 2;
Per Åhgren2507f8c2020-03-19 12:33:29 +0100273 std::array<int16_t, kNumChannels * kSampleRateHz / 100> frame;
274 StreamConfig stream_config(kSampleRateHz, kNumChannels,
275 /*has_keyboard=*/false);
276 frame.fill(kAudioLevel);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100277
278 MockEchoControl* echo_control_mock = echo_control_factory_ptr->GetNext();
279
Sam Zackrisson41478c72019-10-15 10:10:26 +0200280 const int initial_analog_gain = apm->recommended_stream_analog_level();
Per Åhgren0aefbf02019-08-23 21:29:17 +0200281 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100282 EXPECT_CALL(*echo_control_mock, ProcessCapture(NotNull(), testing::_, false))
283 .Times(1);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100284 apm->ProcessStream(frame.data(), stream_config, stream_config, frame.data(),
285 /*vad_result=*/nullptr);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100286
287 // Force an analog gain change if it did not happen.
Sam Zackrisson41478c72019-10-15 10:10:26 +0200288 if (initial_analog_gain == apm->recommended_stream_analog_level()) {
289 apm->set_stream_analog_level(initial_analog_gain + 1);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100290 }
291
Per Åhgren0aefbf02019-08-23 21:29:17 +0200292 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100293 EXPECT_CALL(*echo_control_mock, ProcessCapture(NotNull(), testing::_, true))
294 .Times(1);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100295 apm->ProcessStream(frame.data(), stream_config, stream_config, frame.data(),
296 /*vad_result=*/nullptr);
Alessio Bazzicae4498052018-12-17 09:44:06 +0100297}
298
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200299TEST(AudioProcessingImplTest, EchoControllerObservesPlayoutVolumeChange) {
300 // Tests that the echo controller observes an echo path gain change when a
301 // playout volume change is reported.
Mirko Bonadei317a1f02019-09-17 17:06:18 +0200302 auto echo_control_factory = std::make_unique<MockEchoControlFactory>();
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200303 const auto* echo_control_factory_ptr = echo_control_factory.get();
304
305 std::unique_ptr<AudioProcessing> apm(
306 AudioProcessingBuilder()
307 .SetEchoControlFactory(std::move(echo_control_factory))
308 .Create());
Sam Zackrisson41478c72019-10-15 10:10:26 +0200309 // Disable AGC.
310 webrtc::AudioProcessing::Config apm_config;
311 apm_config.gain_controller1.enabled = false;
312 apm_config.gain_controller2.enabled = false;
313 apm->ApplyConfig(apm_config);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200314
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200315 constexpr int16_t kAudioLevel = 10000;
316 constexpr size_t kSampleRateHz = 48000;
317 constexpr size_t kNumChannels = 2;
Per Åhgren2507f8c2020-03-19 12:33:29 +0100318 std::array<int16_t, kNumChannels * kSampleRateHz / 100> frame;
319 StreamConfig stream_config(kSampleRateHz, kNumChannels,
320 /*has_keyboard=*/false);
321 frame.fill(kAudioLevel);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200322
323 MockEchoControl* echo_control_mock = echo_control_factory_ptr->GetNext();
324
Per Åhgren0aefbf02019-08-23 21:29:17 +0200325 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200326 EXPECT_CALL(*echo_control_mock,
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100327 ProcessCapture(NotNull(), testing::_, /*echo_path_change=*/false))
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200328 .Times(1);
Per Åhgren2507f8c2020-03-19 12:33:29 +0100329 apm->ProcessStream(frame.data(), stream_config, stream_config, frame.data(),
330 /*vad_result=*/nullptr);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200331
Per Åhgren0aefbf02019-08-23 21:29:17 +0200332 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200333 EXPECT_CALL(*echo_control_mock,
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100334 ProcessCapture(NotNull(), testing::_, /*echo_path_change=*/false))
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200335 .Times(1);
336 apm->SetRuntimeSetting(
337 AudioProcessing::RuntimeSetting::CreatePlayoutVolumeChange(50));
Per Åhgren2507f8c2020-03-19 12:33:29 +0100338 apm->ProcessStream(frame.data(), stream_config, stream_config, frame.data(),
339 /*vad_result=*/nullptr);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200340
Per Åhgren0aefbf02019-08-23 21:29:17 +0200341 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200342 EXPECT_CALL(*echo_control_mock,
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100343 ProcessCapture(NotNull(), testing::_, /*echo_path_change=*/false))
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200344 .Times(1);
345 apm->SetRuntimeSetting(
346 AudioProcessing::RuntimeSetting::CreatePlayoutVolumeChange(50));
Per Åhgren2507f8c2020-03-19 12:33:29 +0100347 apm->ProcessStream(frame.data(), stream_config, stream_config, frame.data(),
348 /*vad_result=*/nullptr);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200349
Per Åhgren0aefbf02019-08-23 21:29:17 +0200350 EXPECT_CALL(*echo_control_mock, AnalyzeCapture(testing::_)).Times(1);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200351 EXPECT_CALL(*echo_control_mock,
Per Åhgrenc20a19c2019-11-13 11:12:29 +0100352 ProcessCapture(NotNull(), testing::_, /*echo_path_change=*/true))
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200353 .Times(1);
354 apm->SetRuntimeSetting(
355 AudioProcessing::RuntimeSetting::CreatePlayoutVolumeChange(100));
Per Åhgren2507f8c2020-03-19 12:33:29 +0100356 apm->ProcessStream(frame.data(), stream_config, stream_config, frame.data(),
357 /*vad_result=*/nullptr);
Fredrik Hernqvistca362852019-05-10 15:50:02 +0200358}
359
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200360TEST(AudioProcessingImplTest, RenderPreProcessorBeforeEchoDetector) {
361 // Make sure that signal changes caused by a render pre-processing sub-module
362 // take place before any echo detector analysis.
363 rtc::scoped_refptr<TestEchoDetector> test_echo_detector(
364 new rtc::RefCountedObject<TestEchoDetector>());
365 std::unique_ptr<CustomProcessing> test_render_pre_processor(
366 new TestRenderPreProcessor());
367 // Create APM injecting the test echo detector and render pre-processor.
368 std::unique_ptr<AudioProcessing> apm(
369 AudioProcessingBuilder()
370 .SetEchoDetector(test_echo_detector)
371 .SetRenderPreProcessing(std::move(test_render_pre_processor))
372 .Create());
373 webrtc::AudioProcessing::Config apm_config;
374 apm_config.pre_amplifier.enabled = true;
375 apm_config.residual_echo_detector.enabled = true;
376 apm->ApplyConfig(apm_config);
377
378 constexpr int16_t kAudioLevel = 1000;
379 constexpr int kSampleRateHz = 16000;
380 constexpr size_t kNumChannels = 1;
Sam Zackrisson12e319a2020-01-03 14:54:20 +0100381 // Explicitly initialize APM to ensure no render frames are discarded.
382 const ProcessingConfig processing_config = {{
383 {kSampleRateHz, kNumChannels, /*has_keyboard=*/false},
384 {kSampleRateHz, kNumChannels, /*has_keyboard=*/false},
385 {kSampleRateHz, kNumChannels, /*has_keyboard=*/false},
386 {kSampleRateHz, kNumChannels, /*has_keyboard=*/false},
387 }};
388 apm->Initialize(processing_config);
389
Per Åhgren2507f8c2020-03-19 12:33:29 +0100390 std::array<int16_t, kNumChannels * kSampleRateHz / 100> frame;
391 StreamConfig stream_config(kSampleRateHz, kNumChannels,
392 /*has_keyboard=*/false);
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200393
394 constexpr float kAudioLevelFloat = static_cast<float>(kAudioLevel);
395 constexpr float kExpectedPreprocessedAudioLevel =
396 TestRenderPreProcessor::ProcessSample(kAudioLevelFloat);
397 ASSERT_NE(kAudioLevelFloat, kExpectedPreprocessedAudioLevel);
398
399 // Analyze a render stream frame.
Per Åhgren2507f8c2020-03-19 12:33:29 +0100400 frame.fill(kAudioLevel);
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200401 ASSERT_EQ(AudioProcessing::Error::kNoError,
Per Åhgren2507f8c2020-03-19 12:33:29 +0100402 apm->ProcessReverseStream(frame.data(), stream_config,
403 stream_config, frame.data()));
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200404 // Trigger a call to in EchoDetector::AnalyzeRenderAudio() via
405 // ProcessStream().
Per Åhgren2507f8c2020-03-19 12:33:29 +0100406 frame.fill(kAudioLevel);
407 ASSERT_EQ(AudioProcessing::Error::kNoError,
408 apm->ProcessStream(frame.data(), stream_config, stream_config,
409 frame.data(), /*vad_result=*/nullptr));
Alessio Bazzicad2b97402018-08-09 14:23:11 +0200410 // Regardless of how the call to in EchoDetector::AnalyzeRenderAudio() is
411 // triggered, the line below checks that the call has occurred. If not, the
412 // APM implementation may have changed and this test might need to be adapted.
413 ASSERT_TRUE(test_echo_detector->analyze_render_audio_called());
414 // Check that the data read in EchoDetector::AnalyzeRenderAudio() is that
415 // produced by the render pre-processor.
416 EXPECT_EQ(kExpectedPreprocessedAudioLevel,
417 test_echo_detector->last_render_audio_first_sample());
418}
419
andrew@webrtc.org60730cf2014-01-07 17:45:09 +0000420} // namespace webrtc