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

webrtc/rtc_tools/event_log_visualizer/main.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 70 matching lines @@
 DEFINE_bool(plot_audio_jitter_buffer,
             false,
             "Plot the audio jitter buffer delay profile.");
 DEFINE_string(
     force_fieldtrials,
     "",
     "Field trials control experimental feature code which can be forced. "
     "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(
     show_detector_state,
     false,
     "Mark the delay based bwe detector state on the total bitrate graph");
 
 int main(int argc, char* argv[]) {
   std::string program_name = argv[0];
   std::string usage =
@@ skipping 133 matching lines @@
 
   if (FLAG_plot_all || FLAG_audio_encoder_dtx) {
     analyzer.CreateAudioEncoderEnableDtxGraph(collection->AppendNewPlot());
   }
 
   if (FLAG_plot_all || FLAG_audio_encoder_num_channels) {
     analyzer.CreateAudioEncoderNumChannelsGraph(collection->AppendNewPlot());
   }
 
   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();
 
   return 0;
 }