commit 2f061681cc28daf57a4b904ddebcc2b6ede86d98
author Edward Lemur <ehmaldonado@webrtc.org> Fri Nov 24 12:40:01 2017
committer Commit Bot <commit-bot@chromium.org> Tue Nov 28 11:52:38 2017
tree 7f8a56bcc614c2bcc8aabe53b5a927be06f8fa90
parent e9619f8f81de6ec91b920a0468a6f2c8431bb2f4

Make PrintResultList receive a vector of doubles instead of a string.

Also, add more tests to perf_test_unittest.

Bug: webrtc:8566
Change-Id: I8864db7172fa207803d310c4a5fee4bf820a56bd
Reviewed-on: https://webrtc-review.googlesource.com/25823
Reviewed-by: Fredrik Solenberg <solenberg@webrtc.org>
Reviewed-by: Henrik Lundin <henrik.lundin@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Commit-Queue: Edward Lemur <ehmaldonado@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20906}

call/call_perf_tests.cc

diff --git a/call/call_perf_tests.cc b/call/call_perf_tests.cc
index 88ac2e8..90eaa5a 100644
--- a/call/call_perf_tests.cc
+++ b/call/call_perf_tests.cc
@@ -130,8 +130,7 @@
 
   void PrintResults() {
     test::PrintResultList("stream_offset", "", "synchronization",
-                          test::ValuesToString(sync_offset_ms_list_), "ms",
-                          false);
+                          sync_offset_ms_list_, "ms", false);
   }
 
  private:
@@ -140,7 +139,7 @@
   int64_t first_time_in_sync_;
   rtc::CriticalSection crit_;
   VideoReceiveStream* receive_stream_ RTC_GUARDED_BY(crit_);
-  std::vector<int> sync_offset_ms_list_;
+  std::vector<double> sync_offset_ms_list_;
 };
 
 void CallPerfTest::TestAudioVideoSync(FecMode fec,
@@ -458,8 +457,7 @@
                              "estimated capture NTP time to be "
                              "within bounds.";
       test::PrintResultList("capture_ntp_time", "", "real - estimated",
-                            test::ValuesToString(time_offset_ms_list_), "ms",
-                            true);
+                            time_offset_ms_list_, "ms", true);
     }
 
     rtc::CriticalSection crit_;
@@ -474,7 +472,7 @@
     uint32_t rtp_start_timestamp_;
     typedef std::map<uint32_t, uint32_t> FrameCaptureTimeList;
     FrameCaptureTimeList capture_time_list_ RTC_GUARDED_BY(&crit_);
-    std::vector<int> time_offset_ms_list_;
+    std::vector<double> time_offset_ms_list_;
   } test(net_config, threshold_ms, start_time_ms, run_time_ms);
 
   RunBaseTest(&test);
@@ -651,8 +649,7 @@
           "bitrate_stats_",
           (pad_to_min_bitrate_ ? "min_transmit_bitrate"
                                : "without_min_transmit_bitrate"),
-          "bitrate_kbps", test::ValuesToString(bitrate_kbps_list_), "kbps",
-          false);
+          "bitrate_kbps", bitrate_kbps_list_, "kbps", false);
     }
 
     VideoSendStream* send_stream_;
@@ -661,7 +658,7 @@
     const int min_acceptable_bitrate_;
     const int max_acceptable_bitrate_;
     int num_bitrate_observations_in_range_;
-    std::vector<size_t> bitrate_kbps_list_;
+    std::vector<double> bitrate_kbps_list_;
   } test(pad_to_min_bitrate);
 
   fake_encoder_.SetMaxBitrate(kMaxEncodeBitrateKbps);