From fe9f222c6698d1da080f81b3006398d5789e440a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Per=20=C3=85hgren?= Date: Tue, 26 Sep 2017 23:33:26 +0200 Subject: [PATCH] Reland of Added logging inside AEC3 for render API buffer MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Bug: webrtc:8250 Change-Id: Icd94331237bf5cd0e5aba2644522456184a9eef0 Reviewed-on: https://webrtc-review.googlesource.com/3860 Reviewed-by: Gustaf Ullberg Commit-Queue: Per Åhgren Cr-Commit-Position: refs/heads/master@{#19986} --- modules/audio_processing/aec3/block_processor.cc | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/modules/audio_processing/aec3/block_processor.cc b/modules/audio_processing/aec3/block_processor.cc index 1d4d46a194..1bdd71f308 100644 --- a/modules/audio_processing/aec3/block_processor.cc +++ b/modules/audio_processing/aec3/block_processor.cc @@ -16,6 +16,7 @@ #include "modules/audio_processing/logging/apm_data_dumper.h" #include "rtc_base/atomicops.h" #include "rtc_base/constructormagic.h" +#include "rtc_base/logging.h" namespace webrtc { namespace { @@ -101,11 +102,15 @@ void BlockProcessorImpl::ProcessCapture( // been a render buffer overrun as the buffer alignment may be noncausal. delay_controller_->Reset(); render_buffer_->Reset(); + LOG(LS_WARNING) << "Reset due to detected render buffer overrun."; } // Update the render buffers with new render data, filling the buffers with // empty blocks when there is no render data available. render_buffer_underrun = !render_buffer_->UpdateBuffers(); + if (render_buffer_underrun) { + LOG(LS_WARNING) << "Render API jitter buffer underrun."; + } // Compute and and apply the render delay required to achieve proper signal // alignment. @@ -127,6 +132,7 @@ void BlockProcessorImpl::ProcessCapture( delay_controller_->Reset(); render_buffer_->Reset(); delay_change = true; + LOG(LS_WARNING) << "Reset due to noncausal delay."; } // Remove the echo from the capture signal.