Add aec-refined-adaptive-filter command line switch

content/renderer/media/media_stream_audio_processor.cc

Index: content/renderer/media/media_stream_audio_processor.cc
diff --git a/content/renderer/media/media_stream_audio_processor.cc b/content/renderer/media/media_stream_audio_processor.cc
index 869e41670616676625bcd1c13c154da54d406906..23039de63c527bbc6d704f3489a0adcb08ba90bf 100644
--- a/content/renderer/media/media_stream_audio_processor.cc
+++ b/content/renderer/media/media_stream_audio_processor.cc
@@ -111,6 +111,13 @@ bool GetStartupMinVolumeForAgc(int* startup_min_volume) {
          base::StringToInt(min_volume_str, startup_min_volume);
 }
 
+// Checks if the AEC's refined adaptive filter tuning was enabled on the command
+// line.
+bool UseAecRefinedAdaptiveFilter() {
+  return base::CommandLine::ForCurrentProcess()->HasSwitch(

[aluebs-chromium, 2016/04/18 20:52:49]

Just out of curiosity, does this kind of switches allows you to run a Finch
experiment? Or isn't that the plan?

[hlundin-chromium, 2016/04/19 06:13:35]

The plan is to maybe run a Finch experiment, but not at this stage. For a Finch
experiment, you have to query base::FieldTrialList, I believe.

[aluebs-chromium, 2016/04/19 16:20:51]

Yes, that is what I thought. I was just curious :)

+      switches::kAecRefinedAdaptiveFilter);
+}
+
 }  // namespace
 
 // Wraps AudioBus to provide access to the array of channel pointers, since this
@@ -516,6 +523,10 @@ void MediaStreamAudioProcessor::InitializeAudioProcessingModule(
   config.Set<webrtc::ExperimentalNs>(
       new webrtc::ExperimentalNs(goog_experimental_ns));
   config.Set<webrtc::DelayAgnostic>(new webrtc::DelayAgnostic(true));
+  if (UseAecRefinedAdaptiveFilter()) {
+    config.Set<webrtc::RefinedAdaptiveFilter>(
+        new webrtc::RefinedAdaptiveFilter(true));
+  }

[aluebs-chromium, 2016/04/18 20:52:49]

Do you want to rely on the default WebRTC behavior when the switch is not
specified? Because you could force it to false in the else-clause depending on
the behavior you find better.

[hlundin-chromium, 2016/04/19 06:13:35]

Ack.

I'm relying on the default behavior for now.

[aluebs-chromium, 2016/04/19 16:20:51]

Ack

   if (goog_beamforming) {
     const auto& geometry =
         GetArrayGeometryPreferringConstraints(audio_constraints, input_params);