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

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

Issue 2179223003: Convenience functions to set axis properties in visualization tool. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 5 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/generate_timeseries.cc
diff --git a/webrtc/tools/event_log_visualizer/generate_timeseries.cc b/webrtc/tools/event_log_visualizer/generate_timeseries.cc
index d2139475fa225ee50ce132dd1fd16450d99e1e51..050fbea473d0adc0b0d599b56fe7130f49c1ec35 100644
--- a/webrtc/tools/event_log_visualizer/generate_timeseries.cc
+++ b/webrtc/tools/event_log_visualizer/generate_timeseries.cc
@@ -78,44 +78,44 @@ int main(int argc, char* argv[]) {
if (FLAGS_plot_all || FLAGS_plot_packets) {
if (FLAGS_incoming) {
analyzer.CreatePacketGraph(webrtc::PacketDirection::kIncomingPacket,
- collection->append_new_plot());
+ collection->AppendNewPlot());
}
if (FLAGS_outgoing) {
analyzer.CreatePacketGraph(webrtc::PacketDirection::kOutgoingPacket,
- collection->append_new_plot());
+ collection->AppendNewPlot());
}
}
if (FLAGS_plot_all || FLAGS_plot_audio_playout) {
- analyzer.CreatePlayoutGraph(collection->append_new_plot());
+ analyzer.CreatePlayoutGraph(collection->AppendNewPlot());
}
if (FLAGS_plot_all || FLAGS_plot_sequence_number) {
if (FLAGS_incoming) {
- analyzer.CreateSequenceNumberGraph(collection->append_new_plot());
+ analyzer.CreateSequenceNumberGraph(collection->AppendNewPlot());
}
}
if (FLAGS_plot_all || FLAGS_plot_delay_change) {
if (FLAGS_incoming) {
- analyzer.CreateDelayChangeGraph(collection->append_new_plot());
+ analyzer.CreateDelayChangeGraph(collection->AppendNewPlot());
}
}
if (FLAGS_plot_all || FLAGS_plot_accumulated_delay_change) {
if (FLAGS_incoming) {
- analyzer.CreateAccumulatedDelayChangeGraph(collection->append_new_plot());
+ analyzer.CreateAccumulatedDelayChangeGraph(collection->AppendNewPlot());
}
}
if (FLAGS_plot_all || FLAGS_plot_total_bitrate) {
if (FLAGS_incoming) {
analyzer.CreateTotalBitrateGraph(webrtc::PacketDirection::kIncomingPacket,
- collection->append_new_plot());
+ collection->AppendNewPlot());
}
if (FLAGS_outgoing) {
analyzer.CreateTotalBitrateGraph(webrtc::PacketDirection::kOutgoingPacket,
- collection->append_new_plot());
+ collection->AppendNewPlot());
}
}
@@ -123,16 +123,16 @@ int main(int argc, char* argv[]) {
if (FLAGS_incoming) {
analyzer.CreateStreamBitrateGraph(
webrtc::PacketDirection::kIncomingPacket,
- collection->append_new_plot());
+ collection->AppendNewPlot());
}
if (FLAGS_outgoing) {
analyzer.CreateStreamBitrateGraph(
webrtc::PacketDirection::kOutgoingPacket,
- collection->append_new_plot());
+ collection->AppendNewPlot());
}
}
- collection->draw();
+ collection->Draw();
return 0;
}

Powered by Google App Engine
This is Rietveld 408576698