Throughput: Change the min number of requests in flight required

net/nqe/network_quality_estimator_params.h

 // Copyright 2016 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_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_
 #define NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_
 
 #include <map>
 #include <string>
 
@@ skipping 42 matching lines @@
   const NetworkQuality& DefaultObservation(
       NetworkChangeNotifier::ConnectionType type) const;
 
   // Returns the typical network quality for connection |type|.
   const NetworkQuality& TypicalNetworkQuality(
       EffectiveConnectionType type) const;
 
   // Returns the threshold for effective connection type |type|.
   const NetworkQuality& ConnectionThreshold(EffectiveConnectionType type) const;
 
+  // Returns the minimum number of requests in-flight before the throughput

[RyanSturm, 2017/05/22 20:14:40]

nit: I don't like this phrasing. I'm struggling to come up with a way to say
that this is "the minimum number of requests in-flight to consider the network
fully utilized WRT to throughput observations being recorded".

[tbansal1, 2017/05/22 22:51:55]

Done.

+  // observation can be taken.
+  size_t throughput_min_requests_in_flight() const {
+    return throughput_min_requests_in_flight_;
+  }
+
   // Returns the weight multiplier per second, which represents the factor by
   // which the weight of an observation reduces every second.
   double weight_multiplier_per_second() const {
     return weight_multiplier_per_second_;
   }
 
   // Returns the factor by which the weight of an observation reduces for every
   // dBm difference between the current signal strength (in dBm), and the signal
   // strength at the time when the observation was taken.
   double weight_multiplier_per_dbm() const {
@@ skipping 23 matching lines @@
   // single socket watcher.
   base::TimeDelta min_socket_watcher_notification_interval() const {
     return min_socket_watcher_notification_interval_;
   }
 
  private:
   // Map containing all field trial parameters related to
   // NetworkQualityEstimator field trial.
   const std::map<std::string, std::string> params_;
 
+  const size_t throughput_min_requests_in_flight_;
   const double weight_multiplier_per_second_;
   const double weight_multiplier_per_dbm_;
   const double correlation_uma_logging_probability_;
   const base::Optional<EffectiveConnectionType>
       forced_effective_connection_type_;
   const bool persistent_cache_reading_enabled_;
   const base::TimeDelta min_socket_watcher_notification_interval_;
 
   // Default network quality observations obtained from |params_|.
   NetworkQuality
@@ skipping 15 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(NetworkQualityEstimatorParams);
 };
 
 }  // namespace internal
 
 }  // namespace nqe
 
 }  // namespace net
 
 #endif  // NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_