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

Unified Diff: net/nqe/network_quality_estimator.h

Issue 2322183002: Add Network Quality Estimator (NQE) pref manager (Closed)
Patch Set: Rebased, Addressed kundaji comments Created 4 years, 3 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
Index: net/nqe/network_quality_estimator.h
diff --git a/net/nqe/network_quality_estimator.h b/net/nqe/network_quality_estimator.h
index 248d2089d5ac00583d178efc580ec631920b56bf..38966345be6d80030f3d4a3ce8c360927a7ee87a 100644
--- a/net/nqe/network_quality_estimator.h
+++ b/net/nqe/network_quality_estimator.h
@@ -218,6 +218,14 @@ class NET_EXPORT NetworkQualityEstimator
void ReportEffectiveConnectionTypeForTesting(
EffectiveConnectionType effective_connection_type);
+ // Adds and removes |observer| from the list of cache observers.
+ void AddNetworkQualitiesCacheObserver(
+ nqe::internal::NetworkQualityStore::NetworkQualitiesCacheObserver*
+ observer);
+ void RemoveNetworkQualitiesCacheObserver(
+ nqe::internal::NetworkQualityStore::NetworkQualitiesCacheObserver*
+ observer);
+
protected:
// NetworkChangeNotifier::ConnectionTypeObserver implementation:
void OnConnectionTypeChanged(
@@ -285,9 +293,6 @@ class NET_EXPORT NetworkQualityEstimator
// Returns a random double in the range [0.0, 1.0). Virtualized for testing.
virtual double RandDouble() const;
- // Returns a pointer to |network_quality_store_|. Used only for testing.
- nqe::internal::NetworkQualityStore* NetworkQualityStoreForTesting() const;
-
private:
FRIEND_TEST_ALL_PREFIXES(NetworkQualityEstimatorTest,
AdaptiveRecomputationEffectiveConnectionType);

Powered by Google App Engine
This is Rietveld 408576698