commit dc2242d351c908ac86811c5c9b83a573d98cb717
author peah <peah@webrtc.org> Wed Apr 06 09:30:58 2016 -0700
committer Commit bot <commit-bot@chromium.org> Wed Apr 06 16:31:07 2016 +0000
tree 4e151eddbcda8e82b1a66501d712fc1a757f809b
parent 15c51e355f9ade7fbeb7f7ac6784c0f24446ab7e

Added a query function for whether submodules are enabled
that can be called from the render side without making APM
singlethreaded.

This CL is addressing the problems with high render-side
call duration that were triggered by the CL
https://codereview.webrtc.org/1844583003

BUG=webrtc:5736

Review URL: https://codereview.webrtc.org/1859243002

Cr-Commit-Position: refs/heads/master@{#12266}

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 953b699..0f48fa5 100644
--- a/webrtc/modules/audio_processing/audio_processing_impl.cc
+++ b/webrtc/modules/audio_processing/audio_processing_impl.cc
@@ -1157,9 +1157,11 @@
 bool AudioProcessingImpl::rev_analysis_needed() const {
   return formats_.rev_proc_format.sample_rate_hz() == kSampleRate32kHz &&
          (is_rev_processed() ||
-          public_submodules_->echo_cancellation->is_enabled() ||
-          public_submodules_->echo_control_mobile->is_enabled() ||
-          public_submodules_->gain_control->is_enabled());
+          public_submodules_->echo_cancellation
+              ->is_enabled_render_side_query() ||
+          public_submodules_->echo_control_mobile
+              ->is_enabled_render_side_query() ||
+          public_submodules_->gain_control->is_enabled_render_side_query());
 }
 
 bool AudioProcessingImpl::render_check_rev_conversion_needed() const {