Integrate Intelligibility with APM

webrtc/voice_engine/output_mixer.cc

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 568 matching lines @@
   // side. Downmix to mono.
   AudioFrame frame;
   frame.num_channels_ = 1;
   frame.sample_rate_hz_ = _audioProcessingModulePtr->input_sample_rate_hz();
   RemixAndResample(_audioFrame, &audioproc_resampler_, &frame);
 
   if (_audioProcessingModulePtr->AnalyzeReverseStream(&frame) == -1) {
     WEBRTC_TRACE(kTraceWarning, kTraceVoice, VoEId(_instanceId,-1),
                  "AudioProcessingModule::AnalyzeReverseStream() => error");
   }
+  RemixAndResample(frame, &audioproc_resampler_, &_audioFrame);

[henrika_webrtc, 2015/09/25 10:44:53]

Are we adding complexity for all cases here?

[Andrew MacDonald, 2015/09/26 00:38:32]

Yes, this was a problem, but this is an old patchset and no longer relevant.

 }
 
 int
 OutputMixer::InsertInbandDtmfTone()
 {
     uint16_t sampleRate(0);
     _dtmfGenerator.GetSampleRate(sampleRate);
     if (sampleRate != _audioFrame.sample_rate_hz_)
     {
         // Update sample rate of Dtmf tone since the mixing frequency changed.
@@ skipping 28 matching lines @@
             _audioFrame.data_[2 * i + 1] = 0;
         }
     }
     assert(_audioFrame.samples_per_channel_ == toneSamples);
 
     return 0;
 }
 
 }  // namespace voe
 }  // namespace webrtc