Store network quality samples so we can compute percentiles.

net/base/network_quality.h

Index: net/base/network_quality.h
diff --git a/net/base/network_quality.h b/net/base/network_quality.h
index 4e2985776e0f4a8430b80c7523751391b4b0c0b7..fc85836d42e82fe1950857355007aa8eba342e5b 100644
--- a/net/base/network_quality.h
+++ b/net/base/network_quality.h
@@ -14,36 +14,36 @@ namespace net {
 // API that is used to report the current network quality as estimated by the
 // NetworkQualityEstimator.
 struct NET_EXPORT_PRIVATE NetworkQuality {
-  NetworkQuality(const base::TimeDelta& fastest_rtt,
-                 double fastest_rtt_confidence,
-                 uint64_t peak_throughput_kbps,
-                 double peak_throughput_kbps_confidence)
-      : fastest_rtt(fastest_rtt),
-        fastest_rtt_confidence(fastest_rtt_confidence),
-        peak_throughput_kbps(peak_throughput_kbps),
-        peak_throughput_kbps_confidence(peak_throughput_kbps_confidence) {}
+  NetworkQuality(const base::TimeDelta& rtt,
+                 double rtt_confidence,

[bengr, 2015/06/01 22:55:42]

Define what is meant by each of these parameters.

[tbansal1, 2015/06/02 18:33:01]

Done.

+                 uint64_t throughput_kbps,
+                 double throughput_kbps_confidence)
+      : rtt(rtt),
+        rtt_confidence(rtt_confidence),
+        throughput_kbps(throughput_kbps),
+        throughput_kbps_confidence(throughput_kbps_confidence) {}

[bengr, 2015/06/01 22:55:42]

sanity check all parameters.

[tbansal1, 2015/06/02 18:33:01]

Done.

 
   ~NetworkQuality() {}
 
-  // The fastest round trip time observed for the current connection.
-  const base::TimeDelta fastest_rtt;
+  // The round trip time observed for the current connection.
+  const base::TimeDelta rtt;
 
-  // Confidence of the |fastest_rtt| estimate. Value lies between 0.0 and 1.0
+  // Confidence of the |rtt| estimate. Value lies between 0.0 and 1.0
   // with 0.0 being no confidence and 1.0 implying that estimates are same as
   // ground truth.
   // TODO(tbansal): Define units so values intermediate between 0.0 and 1.0 are
   // meaningful.
-  const double fastest_rtt_confidence;
+  const double rtt_confidence;
 
-  // Peak throughput in Kbps observed for the current connection.
-  const uint64_t peak_throughput_kbps;
+  // Throughput in Kbps observed for the current connection.
+  const uint64_t throughput_kbps;
 
-  // Confidence of the |peak_throughput_kbps| estimate. Value lies between 0.0
+  // Confidence of the |throughput_kbps| estimate. Value lies between 0.0
   // and 1.0 with 0.0 being no confidence and 1.0 implying that estimates are
   // same as ground truth.
   // TODO(tbansal): Define units so values intermediate between 0.0 and 1.0 are
   // meaningful.
-  const double peak_throughput_kbps_confidence;
+  const double throughput_kbps_confidence;
 };
 
 }  // namespace net