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

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

Issue 2695613005: Add ana config to event log visualiser (Closed)
Patch Set: Created 3 years, 10 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 13a5b8a4f9353bc7bf30bd3b6f5425db800fefbc..9e8fe04908ba5ef7297a736f139995cf350338ce 100644
--- a/webrtc/tools/event_log_visualizer/main.cc
+++ b/webrtc/tools/event_log_visualizer/main.cc
@@ -62,6 +62,25 @@ DEFINE_bool(plot_fraction_loss,
DEFINE_bool(plot_timestamps,
false,
"Plot the rtp timestamps of all rtp and rtcp packets over time.");
+DEFINE_bool(audio_encoder_bitrate_bps,
+ false,
+ "Plot the audio encoder target bitrate.");
+DEFINE_bool(audio_encoder_frame_length_ms,
+ false,
+ "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_enable_fec,
+ false,
+ "Plot the audio encoder enable FEC.");
minyue-webrtc 2017/02/14 17:04:22 can remove "enable"
michaelt 2017/02/15 08:06:45 Done.
+DEFINE_bool(audio_encoder_enable_dtx,
+ false,
+ "Plot the audio encoder enable DTX.");
minyue-webrtc 2017/02/14 17:04:22 same here
michaelt 2017/02/15 08:06:46 Done.
+DEFINE_bool(audio_encoder_num_channels,
+ false,
+ "Plot the audio encoder number of channels.");
DEFINE_string(
force_fieldtrials,
"",
@@ -187,6 +206,31 @@ int main(int argc, char* argv[]) {
analyzer.CreateTimestampGraph(collection->AppendNewPlot());
}
+ if (FLAGS_plot_all || FLAGS_audio_encoder_bitrate_bps) {
+ analyzer.CreateAudioEncoderTargetBitrateGraph(collection->AppendNewPlot());
+ }
+
+ if (FLAGS_plot_all || FLAGS_audio_encoder_frame_length_ms) {
+ analyzer.CreateAudioEncoderFrameLengthGraph(collection->AppendNewPlot());
+ }
+
+ if (FLAGS_plot_all || FLAGS_audio_encoder_uplink_packet_loss_fraction) {
+ analyzer.CreateAudioEncoderUplinkPacketLossFractionGraph(
+ collection->AppendNewPlot());
+ }
+
+ if (FLAGS_plot_all || FLAGS_audio_encoder_enable_fec) {
+ analyzer.CreateAudioEncoderEnableFecGraph(collection->AppendNewPlot());
+ }
+
+ if (FLAGS_plot_all || FLAGS_audio_encoder_enable_dtx) {
+ analyzer.CreateAudioEncoderEnableDtxGraph(collection->AppendNewPlot());
+ }
+
+ if (FLAGS_plot_all || FLAGS_audio_encoder_num_channels) {
+ analyzer.CreateAudioEncoderNumChannelsGraph(collection->AppendNewPlot());
+ }
+
collection->Draw();
return 0;

Powered by Google App Engine
This is Rietveld 408576698