Adding APM configuration in AEC dump.

webrtc/modules/audio_processing/debug.proto

Index: webrtc/modules/audio_processing/debug.proto
diff --git a/webrtc/modules/audio_processing/debug.proto b/webrtc/modules/audio_processing/debug.proto
index dce2f7920930452f7a512d9bda2f3679893cddc2..a68b97663f80fc2aa8da4f0153caa1d602ee63df 100644
--- a/webrtc/modules/audio_processing/debug.proto
+++ b/webrtc/modules/audio_processing/debug.proto
@@ -2,6 +2,8 @@ syntax = "proto2";
 option optimize_for = LITE_RUNTIME;
 package webrtc.audioproc;
 
+// Contains the format of input/output/reverse audio. An Init message is added
+// when any of the fields is changed.

[Andrew MacDonald, 2015/10/02 19:04:01]

s/is changed/are changed

 message Init {
   optional int32 sample_rate = 1;
   optional int32 device_sample_rate = 2 [deprecated=true];
@@ -39,11 +41,40 @@ message Stream {
   repeated bytes output_channel = 8;
 }
 
+// Contains the configurations of various APM component. A Config message is
+// added when any of the fields is changed.
+message Config {
+  // Next field number 17.
+  // Acoustic echo canceler.
+  optional bool aec_enabled = 1;
+  optional bool aec_delay_agnostic_enabled = 2;
+  optional bool aec_drift_compensation_enabled = 3;
+  optional bool aec_extended_filter_enabled = 4;
+  optional int32 aec_suppression_level = 5;
+  // Mobile AEC.
+  optional bool aecm_enabled = 6;
+  optional bool aecm_comfort_noise_enabled = 7;
+  optional int32 aecm_routing_mode = 8;
+  // Automatic gain controller.
+  optional bool agc_enabled = 9;
+  optional bool agc_experiment_enabled = 10;

[Andrew MacDonald, 2015/10/02 19:04:02]

Similarly, noise_robust_agc_enabled.

Or something? We haven't really come up with a good name for this thing.

Turaj, Alex, do you have any proposals?

[turajs, 2015/10/02 19:16:36]

I'm normally bad with naming. Don't have a better suggestion than
"noise-robust." 

What differentiates it than other AGCs is its robust Voiced/Unvoiced classifier.
But I don't know how to make a name out of it. It was easy for its internal VAD
to call pitch-based. But I don't think pitch-based-AGC makes much of sense.  I
don't know if I was helpful!

[aluebs-webrtc, 2015/10/02 19:16:37]

I like your suggestion.

[minyue-webrtc, 2015/10/02 19:59:59]

Thanks for your comments. I'd take Andrew's suggestion. A slight change to it is
that I'd like "agc" to be placed as pre-fix, as other names.

[Andrew MacDonald, 2015/10/02 21:41:56]

I get why you want to prefix with agc for consistency, but I'd prefer not in
this case. agc_noise_robust_enabled makes it sound like a feature of the
existing AGC, while noise_robust_agc_enabled makes it sound like a separate
component, which is closer to the truth.

+  optional int32 agc_mode = 11;
+  optional bool agc_limiter_enabled = 12;
+  // High pass filter.
+  optional bool hpf_enabled = 13;
+  // Noise suppression.
+  optional bool ns_enabled = 14;
+  optional bool ns_experiment_enabled = 15;

[Andrew MacDonald, 2015/10/02 19:04:01]

Could you instead call this transient_suppression_enabled? That's what it's
really referring to. The experiment business is legacy.

[minyue-webrtc, 2015/10/02 19:59:59]

Nice, I'd even put it at the end as a parallel to AEC, HPF etc, wdyt?

+  optional int32 ns_level = 16;
+}
+
 message Event {
   enum Type {
     INIT = 0;
     REVERSE_STREAM = 1;
     STREAM = 2;
+    CONFIG = 3;
+    UNKNOWN_EVENT = 4;
   }
 
   required Type type = 1;
@@ -51,4 +82,5 @@ message Event {
   optional Init init = 2;
   optional ReverseStream reverse_stream = 3;
   optional Stream stream = 4;
+  optional Config config = 5;
 }