Convert IntelligibilityEnhancer to multi-threaded mode

webrtc/modules/audio_processing/audio_processing_impl.cc

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 a7c120f3956b13f0c4a798a71f961453979dc213..e96f864672d7a3c4f20b37155dae562775e649dd 100644
--- a/webrtc/modules/audio_processing/audio_processing_impl.cc
+++ b/webrtc/modules/audio_processing/audio_processing_impl.cc
@@ -921,10 +921,6 @@ int AudioProcessingImpl::ProcessReverseStreamLocked() {
   }
 
   if (constants_.intelligibility_enabled) {
-    // Currently run in single-threaded mode when the intelligibility
-    // enhancer is activated.
-    // TODO(peah): Fix to be properly multi-threaded.
-    rtc::CritScope cs(&crit_capture_);
     public_submodules_->intelligibility_enhancer->ProcessRenderAudio(
         ra->split_channels_f(kBand0To8kHz), capture_nonlocked_.split_rate,
         ra->num_channels());
@@ -1235,8 +1231,10 @@ void AudioProcessingImpl::InitializeBeamformer() {
 void AudioProcessingImpl::InitializeIntelligibility() {
   if (constants_.intelligibility_enabled) {
     public_submodules_->intelligibility_enhancer.reset(
-        new IntelligibilityEnhancer(capture_nonlocked_.split_rate,
-                                    render_.render_audio->num_channels()));
+        new IntelligibilityEnhancer(
+            capture_nonlocked_.split_rate,
+            render_.render_audio->num_channels(),
+            public_submodules_->noise_suppression->num_noise_bins()));

[turaj, 2016/03/07 20:34:35]

num_noise_bins() seemed could be declared static, do you think it makes sense to
have it static? 

[aluebs-webrtc, 2016/03/08 10:53:00]

Great point. Done.

   }
 }