Add NetEq delay plotting to event_log_visualizer

webrtc/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 59 matching lines @@
             "Plot the audio encoder frame length.");
 DEFINE_bool(
     audio_encoder_uplink_packet_loss_fraction,
     false,
     "Plot the uplink packet loss fraction which is send to the audio encoder.");
 DEFINE_bool(audio_encoder_fec, false, "Plot the audio encoder FEC.");
 DEFINE_bool(audio_encoder_dtx, false, "Plot the audio encoder DTX.");
 DEFINE_bool(audio_encoder_num_channels,
             false,
             "Plot the audio encoder number of channels.");
+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 \"/\"");
 
 int main(int argc, char* argv[]) {
   std::string program_name = argv[0];
@@ skipping 130 matching lines @@
   }
 
   if (FLAGS_plot_all || FLAGS_audio_encoder_dtx) {
     analyzer.CreateAudioEncoderEnableDtxGraph(collection->AppendNewPlot());
   }
 
   if (FLAGS_plot_all || FLAGS_audio_encoder_num_channels) {
     analyzer.CreateAudioEncoderNumChannelsGraph(collection->AppendNewPlot());
   }
 
+  if (FLAGS_plot_all || FLAGS_plot_audio_jitter_buffer) {

[owb1, 2017/05/17 19:31:39]

I think it should be FLAG instead of FLAGS

[hlundin-webrtc, 2017/05/30 14:56:07]

Not in gflags: https://gflags.github.io/gflags/#using

[ivoc, 2017/05/30 16:29:45]

It turns out someone updated this tool to use base/flags instead of gflags while
you're working on it, so you'll find out when you rebase that Oliver was
actually right ;)

[hlundin-webrtc, 2017/05/31 06:38:33]

Oh, I see. I'll get to the rebase when everything else is done and dusted.

+    analyzer.CreateAudioJitterBufferGraph(
+        "/usr/local/google/home/hlundin/webrtc_src/junk/krefsme48.pcm", 48000,

[ivoc, 2017/05/16 13:25:50]

This looks like it could use an update :-)

[hlundin-webrtc, 2017/05/30 14:56:07]

You think? :)
Done!

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