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

Unified Diff: net/base/host_resolver_impl.cc

Issue 9197009: Adds custom ttl argument to HostCache::Set. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed default TTL from net-internals Created 8 years, 11 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/base/host_resolver_impl.cc
diff --git a/net/base/host_resolver_impl.cc b/net/base/host_resolver_impl.cc
index 217f1e1329e678e0500a337fbc45cf9a33e165e1..12b10165f067c180ddbc1c94f0ad9ab8ff25c0c7 100644
--- a/net/base/host_resolver_impl.cc
+++ b/net/base/host_resolver_impl.cc
@@ -1353,9 +1353,14 @@ void HostResolverImpl::OnJobComplete(Job* job,
RemoveOutstandingJob(job);
// Write result to the cache.
- if (cache_.get())
- cache_->Set(job->key(), net_error, addrlist, base::TimeTicks::Now());
-
+ if (cache_.get()) {
+ base::TimeDelta ttl = base::TimeDelta::FromSeconds(0);
+ if (net_error == OK)
+ ttl = base::TimeDelta::FromMinutes(1);
+ cache_->Set(job->key(), net_error, addrlist,
+ ttl,
+ base::TimeTicks::Now());
+ }
OnJobCompleteInternal(job, net_error, os_error, addrlist);
}

Powered by Google App Engine
This is Rietveld 408576698