Add IntelligibilityEnhancer support to audioproc_float

webrtc/modules/audio_processing/test/audioproc_float.cc

Index: webrtc/modules/audio_processing/test/audioproc_float.cc
diff --git a/webrtc/modules/audio_processing/test/audioproc_float.cc b/webrtc/modules/audio_processing/test/audioproc_float.cc
index b403c1fe05e1c43b84617d63f09e6419732f29e6..7d8a33ba035ae368f42a3c51ed06f487f24d1de9 100644
--- a/webrtc/modules/audio_processing/test/audioproc_float.cc
+++ b/webrtc/modules/audio_processing/test/audioproc_float.cc
@@ -42,10 +42,20 @@ DEFINE_string(
     o,
     "out.wav",
     "Name of the output file to write the processed capture stream to.");
+DEFINE_string(ri, "", "Name of the render input stream file to read from.");
+DEFINE_string(
+    ro,
+    "out_reverse.wav",
+    "Name of the output file to write the processed render stream to.");
 DEFINE_int32(out_channels, 1, "Number of output channels.");
 const bool out_channels_dummy =
     google::RegisterFlagValidator(&FLAGS_out_channels, &ValidateOutChannels);
+DEFINE_int32(rev_out_channels, 1, "Number of reverse output channels.");
+const bool rev_out_channels_dummy =
+    google::RegisterFlagValidator(&FLAGS_rev_out_channels,
+                                  &ValidateOutChannels);
 DEFINE_int32(out_sample_rate, 48000, "Output sample rate in Hz.");
+DEFINE_int32(rev_out_sample_rate, 48000, "Reverse output sample rate in Hz.");
 DEFINE_string(mic_positions, "",
     "Space delimited cartesian coordinates of microphones in meters. "
     "The coordinates of each point are contiguous. "
@@ -77,8 +87,8 @@ const char kUsage[] =
     "an input capture WAV file or protobuf debug dump and writes to an output\n"
     "WAV file.\n"
     "\n"
-    "All components are disabled by default. If any bi-directional components\n"
-    "are enabled, only debug dump files are permitted.";
+    "All components are disabled by default. If the AEC is enabled, only\n"

[peah-webrtc, 2016/03/18 21:57:17]

Actually, I'd rather you fully remove this line about the AEC. It is only valid
due to lines 104-106, and the FLAGS_all setting anyway may turn on the AEC when
operating on wav-files. And apart from not being able to set the proper default
settings for the AEC, I see now reason for why the AEC cannot already be run
using wav-files.

[aluebs-webrtc, 2016/03/18 23:39:29]

Agreed. Done.

+    "debug dump files are permitted.";
 
 }  // namespace
 
@@ -91,13 +101,8 @@ int main(int argc, char* argv[]) {
             "An input file must be specified with either -i or -dump.\n");
     return 1;
   }
-  if (FLAGS_dump.empty() && (FLAGS_aec || FLAGS_ie)) {
-    fprintf(stderr, "-aec and -ie require a -dump file.\n");
-    return 1;
-  }
-  if (FLAGS_ie) {
-    fprintf(stderr,
-            "FIXME(ajm): The intelligibility enhancer output is not dumped.\n");
+  if (FLAGS_dump.empty() && FLAGS_aec) {

[peah-webrtc, 2016/03/18 21:57:17]

Please remove this statement. The statement is not more true for the AEC than
for NS, AGC or any other component. The test program is not at all yet ready to
operate properly on any of those and and I think that it is then better not to
give the impression that it works on wav files for anything but the AEC.

[aluebs-webrtc, 2016/03/18 23:39:29]

Done. If by "not at all yet ready to operate properly" you mean that some
settings are missing, I agree. Or is there something else missing?

[peah-webrtc, 2016/03/19 01:00:16]

There is afaics lots of things missing:
AEC:
set_suppression_level
set_stream_drift_compensation
delay agnostic mode
extended filter mode

AECM
routing mode
comfort noise enabled
set echo path

GainControl
set_stream_analog_level
set_mode
set_target_level_dbfs
set_compression_gain_db
enable_limiter
set_analog_level_limits

I would say that basically all of the submodule-specific settings are missing
apart from setting the noise suppression level which is the only one that is
actually implemented. The settings listed above are the settings that are
publicly available in the APM submodule API and which therefore a test
application must support.

[aluebs-webrtc, 2016/03/19 02:09:55]

Thank you for doing that analysis. I agree we should support these settings. And
I don't think it is a lot of work to do so.

+    fprintf(stderr, "-aec requires a -dump file.\n");
     return 1;
   }
 
@@ -135,8 +140,24 @@ int main(int argc, char* argv[]) {
   if (FLAGS_dump.empty()) {
     auto in_file = std::unique_ptr<WavReader>(new WavReader(FLAGS_i));
     std::cout << FLAGS_i << ": " << in_file->FormatAsString() << std::endl;
-    processor.reset(new WavFileProcessor(std::move(ap), std::move(in_file),
-                                         std::move(out_file)));
+    std::unique_ptr<WavReader> reverse_in_file;
+    std::unique_ptr<WavWriter> reverse_out_file;
+    if (!FLAGS_ri.empty()) {
+      reverse_in_file.reset(new WavReader(FLAGS_ri));
+      reverse_out_file.reset(new WavWriter(
+          FLAGS_ro,
+          FLAGS_rev_out_sample_rate,
+          static_cast<size_t>(FLAGS_rev_out_channels)));
+      std::cout << FLAGS_ri << ": "
+                << reverse_in_file->FormatAsString() << std::endl;
+      std::cout << FLAGS_ro << ": "
+                << reverse_out_file->FormatAsString() << std::endl;
+    }
+    processor.reset(new WavFileProcessor(std::move(ap),
+                                         std::move(in_file),
+                                         std::move(out_file),
+                                         std::move(reverse_in_file),
+                                         std::move(reverse_out_file)));
 
   } else {
     processor.reset(new AecDumpFileProcessor(