Make path to wav file for jitter buffer simulation in event_log_visualizer configurable.

webrtc/rtc_tools/event_log_visualizer/main.cc

Index: webrtc/rtc_tools/event_log_visualizer/main.cc
diff --git a/webrtc/rtc_tools/event_log_visualizer/main.cc b/webrtc/rtc_tools/event_log_visualizer/main.cc
index 7df0374484aec628b85e76f2884ab14054c189e7..8d3a433b0c8b98ecafff82901ce49f540945a42c 100644
--- a/webrtc/rtc_tools/event_log_visualizer/main.cc
+++ b/webrtc/rtc_tools/event_log_visualizer/main.cc
@@ -88,6 +88,9 @@ DEFINE_string(
     "E.g. running with --force_fieldtrials=WebRTC-FooFeature/Enabled/"
     " will assign the group Enabled to field trial WebRTC-FooFeature. Multiple "
     "trials are separated by \"/\"");
+DEFINE_string(wav_filename,
+              "",
+              "Path to wav file used for simulation of jitter buffer");
 DEFINE_bool(help, false, "prints this message");
 
 DEFINE_bool(
@@ -241,11 +244,14 @@ int main(int argc, char* argv[]) {
   }
 
   if (FLAG_plot_all || FLAG_plot_audio_jitter_buffer) {
-    analyzer.CreateAudioJitterBufferGraph(
-        webrtc::test::ResourcePath(
-            "audio_processing/conversational_speech/EN_script2_F_sp2_B1",
-            "wav"),
-        48000, collection->AppendNewPlot());
+    const std::string wav_path =
+        (strlen(FLAG_wav_filename) == 0)
+            ? webrtc::test::ResourcePath(
+                  "audio_processing/conversational_speech/EN_script2_F_sp2_B1",
+                  "wav")
+            : FLAG_wav_filename;
+    analyzer.CreateAudioJitterBufferGraph(FLAG_wav_filename, 48000,

[hlundin-webrtc, 2017/08/15 13:51:17]

This must be a bug. Now you are always using the FLAG value, even if it is
empty, right?

[owb1, 2017/08/16 08:24:05]

You are right. This was a bug. I reverted to the first patch set and left the
check for an empty char array.

+                                          collection->AppendNewPlot());
   }
 
   collection->Draw();