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

Unified Diff: components/domain_reliability/monitor.cc

Issue 691053003: Domain Reliability: Mark beacons from previous networks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
Index: components/domain_reliability/monitor.cc
diff --git a/components/domain_reliability/monitor.cc b/components/domain_reliability/monitor.cc
index d267a923ace009c6d11106984d78850829488dbb..cbcc856212a544c1ac23700ced3e23edf1a83fcd 100644
--- a/components/domain_reliability/monitor.cc
+++ b/components/domain_reliability/monitor.cc
@@ -33,6 +33,7 @@ DomainReliabilityMonitor::DomainReliabilityMonitor(
discard_uploads_set_(false),
weak_factory_(this) {
DCHECK(OnPrefThread());
+ net::NetworkChangeNotifier::AddNetworkChangeObserver(this);
}
DomainReliabilityMonitor::DomainReliabilityMonitor(
@@ -51,6 +52,7 @@ DomainReliabilityMonitor::DomainReliabilityMonitor(
discard_uploads_set_(false),
weak_factory_(this) {
DCHECK(OnPrefThread());
+ net::NetworkChangeNotifier::AddNetworkChangeObserver(this);
davidben 2014/10/30 21:18:00 I think this should be in InitURLRequestContext, o
}
DomainReliabilityMonitor::~DomainReliabilityMonitor() {
@@ -60,6 +62,7 @@ DomainReliabilityMonitor::~DomainReliabilityMonitor() {
DCHECK(OnPrefThread());
ClearContexts();
+ net::NetworkChangeNotifier::RemoveNetworkChangeObserver(this);
}
void DomainReliabilityMonitor::MoveToNetworkThread() {
@@ -144,6 +147,11 @@ void DomainReliabilityMonitor::OnCompleted(net::URLRequest* request,
}
}
+void DomainReliabilityMonitor::OnNetworkChanged(
+ net::NetworkChangeNotifier::ConnectionType type) {
+ last_network_change_time_ = time_->NowTicks();
+}
+
void DomainReliabilityMonitor::ClearBrowsingData(
DomainReliabilityClearMode mode) {
DCHECK(OnNetworkThread());
@@ -218,6 +226,7 @@ DomainReliabilityContext* DomainReliabilityMonitor::AddContext(
new DomainReliabilityContext(time_.get(),
scheduler_params_,
upload_reporter_string_,
+ last_network_change_time_,
&dispatcher_,
uploader_.get(),
config.Pass());
« components/domain_reliability/context_unittest.cc ('K') | « components/domain_reliability/monitor.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698