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

Unified Diff: net/nqe/network_quality_estimator_test_util.h

Issue 2406763003: Add Network Quality observer to NQE (Closed)
Patch Set: Fix presubmit errors (FOR_EACH_OBSERVER is deprecated) Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/nqe/network_quality_estimator.cc ('k') | net/nqe/network_quality_estimator_test_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/nqe/network_quality_estimator_test_util.h
diff --git a/net/nqe/network_quality_estimator_test_util.h b/net/nqe/network_quality_estimator_test_util.h
index 5bad67dffc0348d9a09e4404306f5efd819890b4..5e6399dabeab605acb0878e4425e2f754e51901a 100644
--- a/net/nqe/network_quality_estimator_test_util.h
+++ b/net/nqe/network_quality_estimator_test_util.h
@@ -84,6 +84,18 @@ class TestNetworkQualityEstimator : public NetworkQualityEstimator {
EffectiveConnectionType GetRecentEffectiveConnectionType(
const base::TimeTicks& start_time) const override;
+ // Returns the effective connection type that was set using
+ // |set_effective_connection_type|. If the connection type has not been set,
+ // then the base implementation is called. |http_rtt|, |transport_rtt| and
+ // |downstream_throughput_kbps| are set to the values that were previously
+ // set by calling set_recent_http_rtt(), set_recent_transport_rtt()
+ // and set_recent_transport_rtt() methods, respectively.
+ EffectiveConnectionType GetRecentEffectiveConnectionTypeAndNetworkQuality(
+ const base::TimeTicks& start_time,
+ base::TimeDelta* http_rtt,
+ base::TimeDelta* transport_rtt,
+ int32_t* downstream_throughput_kbps) const override;
+
void set_http_rtt(const base::TimeDelta& http_rtt) {
http_rtt_set_ = true;
http_rtt_ = http_rtt;
« no previous file with comments | « net/nqe/network_quality_estimator.cc ('k') | net/nqe/network_quality_estimator_test_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698