Added an "enable-audio-processor" flag and WebRtcAudioProcessor class

content/renderer/media/webrtc_audio_processor_options.h

Index: content/renderer/media/webrtc_audio_processor_options.h
diff --git a/content/renderer/media/webrtc_audio_processor_options.h b/content/renderer/media/webrtc_audio_processor_options.h
new file mode 100644
index 0000000000000000000000000000000000000000..a85a6e741bf4d6c0937a9fb8635c7e2851c2b634
--- /dev/null
+++ b/content/renderer/media/webrtc_audio_processor_options.h
@@ -0,0 +1,53 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_PROCESSOR_OPTIONS_H_
+#define CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_PROCESSOR_OPTIONS_H_
+
+#include <string>
+
+namespace webrtc {
+
+class AudioFrame;
+class AudioProcessing;
+class MediaConstraintsInterface;
+
+}
+
+namespace content {
+
+using webrtc::AudioProcessing;
+using webrtc::MediaConstraintsInterface;
+
+// Gets the property named by |key| from the |constraints|.
+// Returns true if the key is found and has a valid boolean value; Otherwise
+// false.
+bool GetPropertyFromConstraints(
+    const MediaConstraintsInterface* constraints,
+    const std::string& key);
+
+// Enables the echo cancellation in |audio_processing|.
+void EnableEchoCancellation(AudioProcessing* audio_processing);

[tommi (sloooow) - chröme, 2013/11/21 19:52:03]

why aren't all these methods member functions of AudioProcessing?

[no longer working on chromium, 2013/11/22 13:27:53]

The reason is that this list of methods will grow substantially when we start
supporting more and more constraints. It is definitely better to extract them
out to a helper class instead of making a super long file.

[tommi (sloooow) - chröme, 2013/11/22 14:32:42]

I realize now that AudioProcessing is an interface from third_party/webrtc.  I
was thinking that this was a chrome class and therefore it would make sense to
have these as member methods.

[no longer working on chromium, 2013/11/25 16:36:26]

:) so it is OK to have these methods to be in a separate helper class?
I really want to keep the WebRtcAudioProcessor (or MediaStreamAudioProcessor)
clean and short.

+
+// Enables the noise suppression in |audio_processing|.
+void EnableNoiseSuppression(AudioProcessing* audio_processing);
+
+// Enables the high pass filter in |audio_processing|.
+void EnableHighPassFilter(AudioProcessing* audio_processing);
+
+// Enables the typing detection in |audio_processing|.
+void EnableTypingDetection(AudioProcessing* audio_processing);
+
+// Enables the experimental echo cancellation in |audio_processing|.
+void EnableExperimentalEchoCancellation(AudioProcessing* audio_processing);
+
+// Starts the echo cancellation dump in |audio_processing|.
+void StartAecDump(AudioProcessing* audio_processing);
+
+// Stops the echo cancellation dump in |audio_processing|.
+void StopAecDump(AudioProcessing* audio_processing);
+
+}  // namespace content
+
+#endif  // CONTENT_RENDERER_MEDIA_WEBRTC_AUDIO_PROCESSOR_OPTIONS_H_