commit 218e16d372e05c5d290abe2daa4e23c981f5098d
author peah <peah@webrtc.org> Tue May 31 09:25:16 2016
committer Commit bot <commit-bot@chromium.org> Tue May 31 09:25:24 2016
tree 7684f54a8a93a686f231b3cafd4662f78c6ea908
parent 973d6bfa0ee9c8c2f2a764b320a3bb212e117f28

Added diagnostic AEC debug logpoints for the purpose
of simplifying offline AEC issue analysis.

BUG=

Review-Url: https://codereview.webrtc.org/2016143003
Cr-Original-Commit-Position: refs/heads/master@{#12967}
Cr-Mirrored-From: https://chromium.googlesource.com/external/webrtc
Cr-Mirrored-Commit: 946f36ef397131506756cb79437940d718f5f23a

modules/audio_processing/aec/aec_core.cc

diff --git a/modules/audio_processing/aec/aec_core.cc b/modules/audio_processing/aec/aec_core.cc
index 4c109d7..fb1a7a5 100644
--- a/modules/audio_processing/aec/aec_core.cc
+++ b/modules/audio_processing/aec/aec_core.cc
@@ -827,6 +827,7 @@
   // playout audio volume is low (or even muted) the delay estimation can return
   // a very large delay, which will break the AEC if it is applied.
   if (self->frame_count < kDelayCorrectionStart) {
+    self->data_dumper->DumpRaw("aec_da_reported_delay", 1, &last_delay);
     return 0;
   }
 #endif
@@ -843,6 +844,7 @@
   // 4. Finally, verify that the proposed |delay_correction| is feasible by
   //    comparing with the size of the far-end buffer.
   last_delay = WebRtc_last_delay(self->delay_estimator);
+  self->data_dumper->DumpRaw("aec_da_reported_delay", 1, &last_delay);
   if ((last_delay >= 0) && (last_delay != self->previous_delay) &&
       (WebRtc_last_delay_quality(self->delay_estimator) >
        self->delay_quality_threshold)) {
@@ -888,6 +890,8 @@
              ? delay_quality
              : self->delay_quality_threshold);
   }
+  self->data_dumper->DumpRaw("aec_da_delay_correction", 1, &delay_correction);
+
   return delay_correction;
 }
 
@@ -1154,6 +1158,8 @@
     aec->delayIdx = WebRtcAec_PartitionDelay(aec->num_partitions, aec->wfBuf);
   }
 
+  aec->data_dumper->DumpRaw("aec_nlp_delay", 1, &aec->delayIdx);
+
   // Use delayed far.
   memcpy(xfw, aec->xfwBuf + aec->delayIdx * PART_LEN1,
          sizeof(xfw[0][0]) * 2 * PART_LEN1);
@@ -1173,6 +1179,8 @@
 
   FormSuppressionGain(aec, cohde, cohxd, hNl);
 
+  aec->data_dumper->DumpRaw("aec_nlp_gain", PART_LEN1, hNl);
+
   WebRtcAec_Suppress(hNl, efw);
 
   // Add comfort noise.
@@ -1388,6 +1396,10 @@
                   aec->error_threshold, &x_fft[0][0], &aec->xfBufBlockPos,
                   aec->xfBuf, nearend_ptr, aec->xPow, aec->wfBuf,
                   echo_subtractor_output);
+  aec->data_dumper->DumpRaw("aec_h_fft", PART_LEN1 * aec->num_partitions,
+                            &aec->wfBuf[0][0]);
+  aec->data_dumper->DumpRaw("aec_h_fft", PART_LEN1 * aec->num_partitions,
+                            &aec->wfBuf[1][0]);
 
   aec->data_dumper->DumpWav("aec_out_linear", PART_LEN, echo_subtractor_output,
                             std::min(aec->sampFreq, 16000), 1);