diff --git a/modules/audio_processing/audio_processing_impl.cc b/modules/audio_processing/audio_processing_impl.cc index 6bd03a5a11..3e49c0cb76 100644 --- a/modules/audio_processing/audio_processing_impl.cc +++ b/modules/audio_processing/audio_processing_impl.cc @@ -247,8 +247,7 @@ bool AudioProcessingImpl::SubmoduleStates::RenderMultiBandProcessingActive() bool AudioProcessingImpl::SubmoduleStates::HighPassFilteringRequired() const { return high_pass_filter_enabled_ || echo_canceller_enabled_ || - mobile_echo_controller_enabled_ || noise_suppressor_enabled_ || - echo_controller_enabled_; + mobile_echo_controller_enabled_ || noise_suppressor_enabled_; } AudioProcessingBuilder::AudioProcessingBuilder() = default; @@ -514,7 +513,7 @@ int AudioProcessingImpl::InitializeLocked() { submodules_.agc_manager->SetCaptureMuted(capture_.output_will_be_muted); } InitializeTransient(); - InitializeHighPassFilter(false); + InitializeHighPassFilter(); InitializeVoiceDetector(); InitializeResidualEchoDetector(); InitializeEchoController(); @@ -677,8 +676,7 @@ void AudioProcessingImpl::ApplyConfig(const AudioProcessing::Config& config) { InitializeNoiseSuppressor(); } - InitializeHighPassFilter(config_.echo_canceller.enabled && - !config.echo_canceller.mobile_mode); + InitializeHighPassFilter(); if (agc1_config_changed) { ApplyAgc1Config(config_.gain_controller1); @@ -1790,7 +1788,7 @@ AudioProcessing::Config AudioProcessingImpl::GetConfig() const { bool AudioProcessingImpl::UpdateActiveSubmoduleStates() { return submodule_states_.Update( - config_.high_pass_filter.enabled, config_.echo_canceller.enabled, + config_.high_pass_filter.enabled, !!submodules_.echo_cancellation, !!submodules_.echo_control_mobile, config_.residual_echo_detector.enabled, !!submodules_.legacy_noise_suppressor || !!submodules_.noise_suppressor, submodules_.gain_control->is_enabled(), config_.gain_controller2.enabled, @@ -1809,8 +1807,8 @@ void AudioProcessingImpl::InitializeTransient() { } } -void AudioProcessingImpl::InitializeHighPassFilter(bool force_activation) { - if (force_activation || submodule_states_.HighPassFilteringRequired()) { +void AudioProcessingImpl::InitializeHighPassFilter() { + if (submodule_states_.HighPassFilteringRequired()) { submodules_.high_pass_filter.reset(new HighPassFilter(num_proc_channels())); } else { submodules_.high_pass_filter.reset(); diff --git a/modules/audio_processing/audio_processing_impl.h b/modules/audio_processing/audio_processing_impl.h index f83e5509f0..e5d0573e12 100644 --- a/modules/audio_processing/audio_processing_impl.h +++ b/modules/audio_processing/audio_processing_impl.h @@ -232,8 +232,7 @@ class AudioProcessingImpl : public AudioProcessing { RTC_EXCLUSIVE_LOCKS_REQUIRED(crit_render_, crit_capture_); void InitializeResidualEchoDetector() RTC_EXCLUSIVE_LOCKS_REQUIRED(crit_render_, crit_capture_); - void InitializeHighPassFilter(bool force_activation) - RTC_EXCLUSIVE_LOCKS_REQUIRED(crit_capture_); + void InitializeHighPassFilter() RTC_EXCLUSIVE_LOCKS_REQUIRED(crit_capture_); void InitializeVoiceDetector() RTC_EXCLUSIVE_LOCKS_REQUIRED(crit_capture_); void InitializeEchoController() RTC_EXCLUSIVE_LOCKS_REQUIRED(crit_render_, crit_capture_); diff --git a/modules/audio_processing/test/debug_dump_test.cc b/modules/audio_processing/test/debug_dump_test.cc index 1d3451d0dc..b769a6659a 100644 --- a/modules/audio_processing/test/debug_dump_test.cc +++ b/modules/audio_processing/test/debug_dump_test.cc @@ -51,8 +51,8 @@ class DebugDumpGenerator { bool enable_pre_amplifier); // Constructor that uses default input files. - DebugDumpGenerator(const Config& config, - const AudioProcessing::Config& apm_config); + explicit DebugDumpGenerator(const Config& config, + const AudioProcessing::Config& apm_config); ~DebugDumpGenerator(); @@ -346,7 +346,7 @@ TEST_F(DebugDumpTest, ToggleAec) { generator.StartRecording(); generator.Process(100); - apm_config.echo_canceller.enabled = false; + apm_config.echo_canceller.enabled = true; generator.apm()->ApplyConfig(apm_config); generator.Process(100);