commit 72b7524d878889caa47621d32d3b8a07a5f2fb79
author Sebastian Jansson <srte@webrtc.org> Mon Apr 15 13:10:18 2019
committer Commit Bot <commit-bot@chromium.org> Mon Apr 15 14:47:56 2019
tree 23a76ee98ec7872a8ee07d5b8eb02df3f308d4fe
parent df88cc014a58efb904458a64da7d9ffdc24f7477

Adds more stats to CallStatsCollector.

Also adding checks to avoid adding empty stats.

Bug: webrtc:10365
Change-Id: I37ab32a3d4271fcad419f17841a8d2e524d73245
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/133020
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27625}

test/scenario/performance_stats.h

diff --git a/test/scenario/performance_stats.h b/test/scenario/performance_stats.h
index 1979876..429d60f 100644
--- a/test/scenario/performance_stats.h
+++ b/test/scenario/performance_stats.h
@@ -134,6 +134,8 @@
 
 struct CollectedCallStats {
   SampleStats<DataRate> target_rate;
+  SampleStats<TimeDelta> pacer_delay;
+  SampleStats<TimeDelta> round_trip_time;
   SampleStats<double> memory_usage;
 };
 

test/scenario/stats_collection.cc

diff --git a/test/scenario/stats_collection.cc b/test/scenario/stats_collection.cc
index 104c824..817d947 100644
--- a/test/scenario/stats_collection.cc
+++ b/test/scenario/stats_collection.cc
@@ -112,7 +112,12 @@
 }
 
 void CallStatsCollector::AddStats(Call::Stats sample) {
-  stats_.target_rate.AddSampleBps(sample.send_bandwidth_bps);
+  if (sample.send_bandwidth_bps > 0)
+    stats_.target_rate.AddSampleBps(sample.send_bandwidth_bps);
+  if (sample.pacer_delay_ms > 0)
+    stats_.pacer_delay.AddSample(TimeDelta::ms(sample.pacer_delay_ms));
+  if (sample.rtt_ms > 0)
+    stats_.round_trip_time.AddSample(TimeDelta::ms(sample.rtt_ms));
   stats_.memory_usage.AddSample(rtc::GetProcessResidentSizeBytes());
 }