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

Unified Diff: webrtc/modules/audio_processing/audio_processing_impl.cc

Issue 1803433003: Removed the dependency on AudioProcessingImpl in EchoControlMobileImpl (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase Created 4 years, 9 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 | « no previous file | webrtc/modules/audio_processing/echo_control_mobile_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/audio_processing/audio_processing_impl.cc
diff --git a/webrtc/modules/audio_processing/audio_processing_impl.cc b/webrtc/modules/audio_processing/audio_processing_impl.cc
index 32ebb0a7c054401cdfad1946de9800deff7c4f75..367586910dc73fc07c362bc171aea30112422349 100644
--- a/webrtc/modules/audio_processing/audio_processing_impl.cc
+++ b/webrtc/modules/audio_processing/audio_processing_impl.cc
@@ -170,7 +170,7 @@ AudioProcessingImpl::AudioProcessingImpl(const Config& config,
public_submodules_->echo_cancellation.reset(
new EchoCancellationImpl(this, &crit_render_, &crit_capture_));
public_submodules_->echo_control_mobile.reset(
- new EchoControlMobileImpl(this, &crit_render_, &crit_capture_));
+ new EchoControlMobileImpl(&crit_render_, &crit_capture_));
public_submodules_->gain_control.reset(
new GainControlImpl(&crit_capture_, &crit_capture_));
public_submodules_->high_pass_filter.reset(
@@ -706,8 +706,17 @@ int AudioProcessingImpl::ProcessStreamLocked() {
public_submodules_->intelligibility_enhancer->SetCaptureNoiseEstimate(
public_submodules_->noise_suppression->NoiseEstimate());
}
- RETURN_ON_ERR(
- public_submodules_->echo_control_mobile->ProcessCaptureAudio(ca));
+
+ // Ensure that the stream delay was set before the call to the
+ // AECM ProcessCaptureAudio function.
+ if (public_submodules_->echo_control_mobile->is_enabled() &&
+ !was_stream_delay_set()) {
+ return AudioProcessing::kStreamParameterNotSetError;
+ }
+
+ RETURN_ON_ERR(public_submodules_->echo_control_mobile->ProcessCaptureAudio(
+ ca, stream_delay_ms()));
+
public_submodules_->voice_detection->ProcessCaptureAudio(ca);
if (constants_.use_experimental_agc &&
@@ -1227,7 +1236,8 @@ void AudioProcessingImpl::InitializeGainController() {
}
void AudioProcessingImpl::InitializeEchoControlMobile() {
- public_submodules_->echo_control_mobile->Initialize();
+ public_submodules_->echo_control_mobile->Initialize(
+ proc_sample_rate_hz(), num_reverse_channels(), num_output_channels());
}
void AudioProcessingImpl::InitializeLevelEstimator() {
« no previous file with comments | « no previous file | webrtc/modules/audio_processing/echo_control_mobile_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698