Pass label and plot style to the TimeSeries' constructor.

webrtc/tools/event_log_visualizer/plot_base.h

 /*
  *  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.
  */
 #ifndef WEBRTC_TOOLS_EVENT_LOG_VISUALIZER_PLOT_BASE_H_
@@ skipping 10 matching lines @@
 enum PlotStyle { LINE_GRAPH, LINE_DOT_GRAPH, BAR_GRAPH, LINE_STEP_GRAPH };
 
 struct TimeSeriesPoint {
   TimeSeriesPoint(float x, float y) : x(x), y(y) {}
   float x;
   float y;
 };
 
 struct TimeSeries {
   TimeSeries() = default;
+  TimeSeries(const char* label, PlotStyle style)
+      : label(label), style(style), points() {}
+  TimeSeries(const std::string& label, PlotStyle style)
+      : label(label), style(style), points() {}
   TimeSeries(TimeSeries&& other)
       : label(std::move(other.label)),
         style(other.style),
         points(std::move(other.points)) {}
   TimeSeries& operator=(TimeSeries&& other) {
     label = std::move(other.label);
     style = other.style;
     points = std::move(other.points);
     return *this;
   }
@@ skipping 49 matching lines @@
   // y-values and are added to either side of the plot.
   void SetSuggestedYAxis(float min_value,
                          float max_value,
                          std::string label,
                          float bottom_margin = 0,
                          float top_margin = 0);
 
   // Sets the title of the plot.
   void SetTitle(std::string title);
 
+  // Add a new TimeSeries to the plot.
+  TimeSeries* AddTimeSeries(const char* label, PlotStyle style) {

[michaelt, 2017/03/13 15:36:42]

Shouldn't we code this in the cc file ?

[terelius, 2017/03/14 17:13:59]

Yes. Done.

+    series_list_.emplace_back(label, style);
+    return &series_list_.back();
+  }
+
+  // Add a new TimeSeries to the plot.
+  TimeSeries* AddTimeSeries(const std::string& label, PlotStyle style) {
+    series_list_.emplace_back(label, style);
+    return &series_list_.back();
+  }
+
   std::vector<TimeSeries> series_list_;
 
  protected:
   float xaxis_min_;
   float xaxis_max_;
   std::string xaxis_label_;
   float yaxis_min_;
   float yaxis_max_;
   std::string yaxis_label_;
   std::string title_;
 };
 
 class PlotCollection {
  public:
   virtual ~PlotCollection() {}
   virtual void Draw() = 0;
   virtual Plot* AppendNewPlot() = 0;
 
  protected:
   std::vector<std::unique_ptr<Plot> > plots_;
 };
 
 }  // namespace plotting
 }  // namespace webrtc
 
 #endif  // WEBRTC_TOOLS_EVENT_LOG_VISUALIZER_PLOT_BASE_H_