commit 135259ac8fac1fd88e948742e6ab473425a43d88
author peah <peah@webrtc.org> Fri Oct 28 03:12:11 2016 -0700
committer Commit bot <commit-bot@chromium.org> Fri Oct 28 10:12:15 2016 +0000
tree a523fdc37f2c6947e08fc94168772d290f6bf429
parent 04055e95bf97d106053d90bcc9e974eb4ad175e6

In order to be able to analyze the AGC behavior on
aecdump recordings in an efficient manner, it is
important to be able to use a standardized analysis
script. For this to be feasible, data log points should
be present.

This CL adds those logpoints as well as the framework
needed to for those to work.

BUG=webrtc:6564

Review-Url: https://codereview.webrtc.org/2457783003
Cr-Commit-Position: refs/heads/master@{#14812}

webrtc/modules/audio_processing/gain_control_impl.cc

diff --git a/webrtc/modules/audio_processing/gain_control_impl.cc b/webrtc/modules/audio_processing/gain_control_impl.cc
index 8f707fc..81469dd 100644
--- a/webrtc/modules/audio_processing/gain_control_impl.cc
+++ b/webrtc/modules/audio_processing/gain_control_impl.cc
@@ -14,6 +14,7 @@
 #include "webrtc/base/optional.h"
 #include "webrtc/modules/audio_processing/audio_buffer.h"
 #include "webrtc/modules/audio_processing/agc/legacy/gain_control.h"
+#include "webrtc/modules/audio_processing/logging/apm_data_dumper.h"
 
 namespace webrtc {
 
@@ -84,10 +85,13 @@
   RTC_DISALLOW_COPY_AND_ASSIGN(GainController);
 };
 
+int GainControlImpl::instance_counter_ = 0;
+
 GainControlImpl::GainControlImpl(rtc::CriticalSection* crit_render,
                                  rtc::CriticalSection* crit_capture)
     : crit_render_(crit_render),
       crit_capture_(crit_capture),
+      data_dumper_(new ApmDataDumper(instance_counter_)),
       mode_(kAdaptiveAnalog),
       minimum_capture_level_(0),
       maximum_capture_level_(255),
@@ -239,6 +243,7 @@
 // TODO(ajm): ensure this is called under kAdaptiveAnalog.
 int GainControlImpl::set_stream_analog_level(int level) {
   rtc::CritScope cs(crit_capture_);
+  data_dumper_->DumpRaw("gain_control_set_stream_analog_level", 1, &level);
 
   was_analog_level_set_ = true;
   if (level < minimum_capture_level_ || level > maximum_capture_level_) {
@@ -251,6 +256,8 @@
 
 int GainControlImpl::stream_analog_level() {
   rtc::CritScope cs(crit_capture_);
+  data_dumper_->DumpRaw("gain_control_stream_analog_level", 1,
+                        &analog_capture_level_);
   // TODO(ajm): enable this assertion?
   //RTC_DCHECK_EQ(kAdaptiveAnalog, mode_);
 
@@ -385,6 +392,7 @@
 void GainControlImpl::Initialize(size_t num_proc_channels, int sample_rate_hz) {
   rtc::CritScope cs_render(crit_render_);
   rtc::CritScope cs_capture(crit_capture_);
+  data_dumper_->InitiateNewSetOfRecordings();
 
   num_proc_channels_ = rtc::Optional<size_t>(num_proc_channels);
   sample_rate_hz_ = rtc::Optional<int>(sample_rate_hz);