Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(288)

Unified Diff: webrtc/tools/event_log_visualizer/main.cc

Issue 2876423002: Add NetEq delay plotting to event_log_visualizer (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/tools/event_log_visualizer/main.cc
diff --git a/webrtc/tools/event_log_visualizer/main.cc b/webrtc/tools/event_log_visualizer/main.cc
index 2f5ecd63e45f67bc4e12ac4910d5593393c57660..844b2360c965811fed6d8b3887edf022a896a67c 100644
--- a/webrtc/tools/event_log_visualizer/main.cc
+++ b/webrtc/tools/event_log_visualizer/main.cc
@@ -77,6 +77,9 @@ 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,
"",
@@ -227,6 +230,12 @@ int main(int argc, char* argv[]) {
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/#us
ivoc 2017/05/30 16:29:45 It turns out someone updated this tool to use base
hlundin-webrtc 2017/05/31 06:38:33 Oh, I see. I'll get to the rebase when everything
+ 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;

Powered by Google App Engine
This is Rietveld 408576698