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..241d11f12c02fcbdb64d9d4eb27c02cca57ec377 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 are changed.
 message Init {
   optional int32 sample_rate = 1;
   optional int32 device_sample_rate = 2 [deprecated=true];
@@ -39,11 +41,41 @@ 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.

[Andrew MacDonald, 2015/10/02 21:46:04]

s/is/are

[minyue-webrtc, 2015/10/02 21:57:34]

oh, thanks. I did not realize that I had made a similar error as in line 6.

+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_noise_robust_enabled = 10;
+  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 int32 ns_level = 15;
+  // Transcient suppression.

[Andrew MacDonald, 2015/10/02 21:46:04]

s/Transcient/Transient

But also a pretty useless comment :)

[minyue-webrtc, 2015/10/02 21:57:34]

Oh, sorry for the typo

+  optional bool transient_suppression_enabled = 16;

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

not sure if it is possible/better to use a short name for transient_suppression
as in other fields.

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

That would be nice, but it's not really a recognized name so I think we'd better
spell it out.

[minyue-webrtc, 2015/10/02 21:57:34]

Acknowledged.

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