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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/nqe/network_quality_estimator.h

Issue 2369673004: Wire NQE Prefs to Profile (Closed)
Patch Set: Addressed ryansturm comments 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
« net/nqe/network_qualities_prefs_manager.cc ('K') | « net/nqe/network_qualities_prefs_manager_unittest.cc ('k') | net/nqe/network_quality_estimator.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/nqe/network_quality_estimator.h
diff --git a/net/nqe/network_quality_estimator.h b/net/nqe/network_quality_estimator.h
index 46b6a8f5017a82ffc38f73aad70945f38fccf93f..9e01001d8e7b8d4e954a096afe9da222572dc47d 100644
--- a/net/nqe/network_quality_estimator.h
+++ b/net/nqe/network_quality_estimator.h
@@ -241,6 +241,13 @@ class NET_EXPORT NetworkQualityEstimator
nqe::internal::NetworkQualityStore::NetworkQualitiesCacheObserver*
observer);
+ // Notifies |this| when the persistent prefs have been read. |read_prefs|
RyanSturm 2016/10/14 18:36:21 nit: s/Notifies |this| when/Called when
tbansal1 2016/10/14 21:15:10 Done.
+ // contains the parsed prefs as a map between NetworkIDs and
+ // CachedNetworkQualities.
+ void OnPrefsRead(
+ const std::map<nqe::internal::NetworkID,
+ nqe::internal::CachedNetworkQuality> read_prefs);
+
protected:
// NetworkChangeNotifier::ConnectionTypeObserver implementation:
void OnConnectionTypeChanged(
« net/nqe/network_qualities_prefs_manager.cc ('K') | « net/nqe/network_qualities_prefs_manager_unittest.cc ('k') | net/nqe/network_quality_estimator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698