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

Unified Diff: net/nqe/network_quality_store.cc

Issue 2261813002: Add a network quality cache observer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: PS Created 4 years, 4 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_quality_store.h ('K') | « net/nqe/network_quality_store.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/nqe/network_quality_store.cc
diff --git a/net/nqe/network_quality_store.cc b/net/nqe/network_quality_store.cc
index 4f744917070899451e83b64ae507b4ad0527fb0d..d985233b1ce4858e232a6ca3c7e8b08a46848ccb 100644
--- a/net/nqe/network_quality_store.cc
+++ b/net/nqe/network_quality_store.cc
@@ -59,6 +59,11 @@ void NetworkQualityStore::Add(
std::make_pair(network_id, cached_network_quality));
DCHECK_LE(cached_network_qualities_.size(),
static_cast<size_t>(kMaximumNetworkQualityCacheSize));
+
+ FOR_EACH_OBSERVER(
+ NetworkQualitiesCacheObserver,
+ network_qualities_persistent_caching_observer_list_,
+ OnChangeInCachedNetworkQuality(network_id, cached_network_quality));
}
bool NetworkQualityStore::GetById(
@@ -76,6 +81,20 @@ bool NetworkQualityStore::GetById(
return true;
}
+void NetworkQualityStore::AddNetworkQualitiesCacheObserver(
+ nqe::internal::NetworkQualityStore::NetworkQualitiesCacheObserver*
+ observer) {
+ DCHECK(thread_checker_.CalledOnValidThread());
+ network_qualities_persistent_caching_observer_list_.AddObserver(observer);
+}
+
+void NetworkQualityStore::RemoveNetworkQualitiesCacheObserver(
+ nqe::internal::NetworkQualityStore::NetworkQualitiesCacheObserver*
+ observer) {
+ DCHECK(thread_checker_.CalledOnValidThread());
+ network_qualities_persistent_caching_observer_list_.RemoveObserver(observer);
+}
+
} // namespace internal
} // namespace nqe
« net/nqe/network_quality_store.h ('K') | « net/nqe/network_quality_store.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698