Adding APM configuration in AEC dump.

webrtc/modules/audio_processing/echo_cancellation_impl.h

Index: webrtc/modules/audio_processing/echo_cancellation_impl.h
diff --git a/webrtc/modules/audio_processing/echo_cancellation_impl.h b/webrtc/modules/audio_processing/echo_cancellation_impl.h
index 9c2b32c473bb6743d116a513b223aa23bcecb0a3..ba63eb54e71189560a5fa949c9cb90972f3499fc 100644
--- a/webrtc/modules/audio_processing/echo_cancellation_impl.h
+++ b/webrtc/modules/audio_processing/echo_cancellation_impl.h
@@ -37,20 +37,22 @@ class EchoCancellationImpl : public EchoCancellation,
   int Initialize() override;
   void SetExtraOptions(const Config& config) override;
 
+  SuppressionLevel suppression_level() const override;

[peah-webrtc, 2015/09/22 12:05:29]

Is it really needed to change the public/private statuses of the member
functions? These are public in the EchoCancellation parent class and they are
publicly accessed from elsewhere so I don't think this api change is needed.

[minyue-webrtc, 2015/09/22 13:19:29]

The problem is the instantiate of echo canceler in APM is an
EchoCancellationImpl

[peah-webrtc, 2015/09/24 09:28:02]

After a discussion together we agreed that to comply with the ongoing locking
refactoring the submodules apis needs to anyway change according to this so it
is motivated to do the change you propose. And the mismatch in public/private
could as well be fixed at the same time.

[peah-webrtc, 2015/09/24 09:28:02]

Acknowledged.

+  bool is_drift_compensation_enabled() const override;
+  bool is_delay_logging_enabled() const override;
+  bool is_extended_filter_enabled() const;
+
  private:
   // EchoCancellation implementation.
   int Enable(bool enable) override;
   int enable_drift_compensation(bool enable) override;
-  bool is_drift_compensation_enabled() const override;
   void set_stream_drift_samples(int drift) override;
   int set_suppression_level(SuppressionLevel level) override;
-  SuppressionLevel suppression_level() const override;
   int enable_metrics(bool enable) override;
   bool are_metrics_enabled() const override;
   bool stream_has_echo() const override;
   int GetMetrics(Metrics* metrics) override;
   int enable_delay_logging(bool enable) override;
-  bool is_delay_logging_enabled() const override;
   int GetDelayMetrics(int* median, int* std) override;
   int GetDelayMetrics(int* median,
                       int* std,