Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(612)

Side by Side Diff: net/nqe/network_quality_estimator_params.h

Issue 2893933004: Throughput: Change the min number of requests in flight required (Closed)
Patch Set: ryansturm comments Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « net/nqe/network_quality_estimator.cc ('k') | net/nqe/network_quality_estimator_params.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_ 5 #ifndef NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_
6 #define NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_ 6 #define NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 const NetworkQuality& DefaultObservation( 53 const NetworkQuality& DefaultObservation(
54 NetworkChangeNotifier::ConnectionType type) const; 54 NetworkChangeNotifier::ConnectionType type) const;
55 55
56 // Returns the typical network quality for connection |type|. 56 // Returns the typical network quality for connection |type|.
57 const NetworkQuality& TypicalNetworkQuality( 57 const NetworkQuality& TypicalNetworkQuality(
58 EffectiveConnectionType type) const; 58 EffectiveConnectionType type) const;
59 59
60 // Returns the threshold for effective connection type |type|. 60 // Returns the threshold for effective connection type |type|.
61 const NetworkQuality& ConnectionThreshold(EffectiveConnectionType type) const; 61 const NetworkQuality& ConnectionThreshold(EffectiveConnectionType type) const;
62 62
63 // Returns the minimum number of requests in-flight to consider the network
64 // fully utilized. A throughput observation is taken only when the network is
65 // considered as fully utilized.
66 size_t throughput_min_requests_in_flight() const {
67 return throughput_min_requests_in_flight_;
68 }
69
63 // Returns the weight multiplier per second, which represents the factor by 70 // Returns the weight multiplier per second, which represents the factor by
64 // which the weight of an observation reduces every second. 71 // which the weight of an observation reduces every second.
65 double weight_multiplier_per_second() const { 72 double weight_multiplier_per_second() const {
66 return weight_multiplier_per_second_; 73 return weight_multiplier_per_second_;
67 } 74 }
68 75
69 // Returns the factor by which the weight of an observation reduces for every 76 // Returns the factor by which the weight of an observation reduces for every
70 // dBm difference between the current signal strength (in dBm), and the signal 77 // dBm difference between the current signal strength (in dBm), and the signal
71 // strength at the time when the observation was taken. 78 // strength at the time when the observation was taken.
72 double weight_multiplier_per_dbm() const { 79 double weight_multiplier_per_dbm() const {
(...skipping 23 matching lines...) Expand all
96 // single socket watcher. 103 // single socket watcher.
97 base::TimeDelta min_socket_watcher_notification_interval() const { 104 base::TimeDelta min_socket_watcher_notification_interval() const {
98 return min_socket_watcher_notification_interval_; 105 return min_socket_watcher_notification_interval_;
99 } 106 }
100 107
101 private: 108 private:
102 // Map containing all field trial parameters related to 109 // Map containing all field trial parameters related to
103 // NetworkQualityEstimator field trial. 110 // NetworkQualityEstimator field trial.
104 const std::map<std::string, std::string> params_; 111 const std::map<std::string, std::string> params_;
105 112
113 const size_t throughput_min_requests_in_flight_;
106 const double weight_multiplier_per_second_; 114 const double weight_multiplier_per_second_;
107 const double weight_multiplier_per_dbm_; 115 const double weight_multiplier_per_dbm_;
108 const double correlation_uma_logging_probability_; 116 const double correlation_uma_logging_probability_;
109 const base::Optional<EffectiveConnectionType> 117 const base::Optional<EffectiveConnectionType>
110 forced_effective_connection_type_; 118 forced_effective_connection_type_;
111 const bool persistent_cache_reading_enabled_; 119 const bool persistent_cache_reading_enabled_;
112 const base::TimeDelta min_socket_watcher_notification_interval_; 120 const base::TimeDelta min_socket_watcher_notification_interval_;
113 121
114 // Default network quality observations obtained from |params_|. 122 // Default network quality observations obtained from |params_|.
115 NetworkQuality 123 NetworkQuality
(...skipping 15 matching lines...) Expand all
131 DISALLOW_COPY_AND_ASSIGN(NetworkQualityEstimatorParams); 139 DISALLOW_COPY_AND_ASSIGN(NetworkQualityEstimatorParams);
132 }; 140 };
133 141
134 } // namespace internal 142 } // namespace internal
135 143
136 } // namespace nqe 144 } // namespace nqe
137 145
138 } // namespace net 146 } // namespace net
139 147
140 #endif // NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_ 148 #endif // NET_NQE_NETWORK_QUALITY_ESTIMATOR_PARAMS_H_
OLDNEW
« no previous file with comments | « net/nqe/network_quality_estimator.cc ('k') | net/nqe/network_quality_estimator_params.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698