From 19f5be3e7b77eccf51148a3ded9944d739aba197 Mon Sep 17 00:00:00 2001 From: Bjorn Terelius Date: Wed, 18 Oct 2017 12:39:49 +0200 Subject: [PATCH] Fix bug that causes the visualization of RTCP fraction lost to be cropped. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The bug comes from computing the axis limits before adding the data. Bug: webrtc:8417 Change-Id: I1e3860b4f655f40cd261d09b8357d563dbe51f1d Reviewed-on: https://webrtc-review.googlesource.com/12922 Reviewed-by: Philip Eliasson Commit-Queue: Björn Terelius Cr-Commit-Position: refs/heads/master@{#20355} --- rtc_tools/event_log_visualizer/analyzer.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rtc_tools/event_log_visualizer/analyzer.cc b/rtc_tools/event_log_visualizer/analyzer.cc index 562a959fef..0325c68f8c 100644 --- a/rtc_tools/event_log_visualizer/analyzer.cc +++ b/rtc_tools/event_log_visualizer/analyzer.cc @@ -898,11 +898,11 @@ void EventLogAnalyzer::CreateFractionLossGraph(Plot* plot) { time_series.points.emplace_back(x, y); } + plot->AppendTimeSeries(std::move(time_series)); plot->SetXAxis(0, call_duration_s_, "Time (s)", kLeftMargin, kRightMargin); plot->SetSuggestedYAxis(0, 10, "Percent lost packets", kBottomMargin, kTopMargin); plot->SetTitle("Reported packet loss"); - plot->AppendTimeSeries(std::move(time_series)); } // Plot the total bandwidth used by all RTP streams.