Store network quality samples so we can compute percentiles.

net/base/network_quality.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_BASE_NETWORK_QUALITY_H_
 #define NET_BASE_NETWORK_QUALITY_H_
 
 #include <stdint.h>
 
 #include "base/time/time.h"
+#include "net/base/net_export.h"
 
 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) {}
+class NET_EXPORT_PRIVATE NetworkQuality {
+ public:
+  // |rtt| is the estimate of the round trip time for the current connection.
+  // |downstream_throughput_kbps| is the estimate of the downstram throughput
+  // for the current connection.
+  NetworkQuality(const base::TimeDelta& rtt,
+                 int32_t downstream_throughput_kbps);
 
-  ~NetworkQuality() {}
+  ~NetworkQuality();
 
-  // The fastest round trip time observed for the current connection.
-  const base::TimeDelta fastest_rtt;
+  // Returns the round trip time observed for the current connection.

[bengr, 2015/06/09 20:46:33]

You could add the clause "for the current connection" to the class description,
after "the current network quality". Then you could remove that clause here and
below.

[tbansal1, 2015/06/10 01:00:05]

Actually, there is nothing that says NetworkQuality class is for
"current" networks only. So, I removed "current" from the comments.
Also, followed on your suggestion to simplify the comments. 

+  base::TimeDelta rtt() const { return rtt_; }

[bengr, 2015/06/09 20:46:32]

Should this return a const&?

[tbansal1, 2015/06/10 01:00:05]

Done.

 
-  // Confidence of the |fastest_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;
+  // Returns the downstream throughput in Kbps observed for the current
+  // connection.
+  int32_t downstream_throughput_kbps() const {
+    return downstream_throughput_kbps_;
+  }
 
-  // Peak throughput in Kbps observed for the current connection.
-  const uint64_t peak_throughput_kbps;
+ private:
+  // The round trip time observed for the current connection.
+  const base::TimeDelta rtt_;
 
-  // Confidence of the |peak_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;
+  // Downstream throughput in Kbps observed for the current connection.
+  const int32_t downstream_throughput_kbps_;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NETWORK_QUALITY_H_