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

Unified Diff: net/dns/host_resolver_mojo.cc

Issue 1928743003: DNS: Support reprioritization of requests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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/dns/host_resolver_impl_unittest.cc ('K') | « net/dns/host_resolver_mojo.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/dns/host_resolver_mojo.cc
diff --git a/net/dns/host_resolver_mojo.cc b/net/dns/host_resolver_mojo.cc
index 1daf6338fd425bffd0ad1db3a6497b3f5d164325..d65a81e1cde63a5b0bd88b5e5727cfc18a0f2532 100644
--- a/net/dns/host_resolver_mojo.cc
+++ b/net/dns/host_resolver_mojo.cc
@@ -92,6 +92,11 @@ int HostResolverMojo::ResolveFromCache(const RequestInfo& info,
return ResolveFromCacheInternal(info, CacheKeyForRequest(info), addresses);
}
+void HostResolverMojo::ChangeRequestPriority(RequestHandle req,
+ RequestPriority priority) {
+ // Do nothing, since Resolve() discarded the priority anyway.
Randy Smith (Not in Mondays) 2016/04/28 18:04:40 Oh, for goodness sake. What uses the DNS resolver
Julia Tuttle 2016/05/04 14:43:20 No idea!
mmenke 2016/05/04 17:33:16 The out of process PAC stuff, I would assume. I b
Randy Smith (Not in Mondays) 2016/05/05 00:07:25 Matt's probably right. Willing to put a TODO in t
Julia Tuttle 2016/05/05 12:47:56 I put a note in the .h instead, where I think a us
mmenke 2016/05/05 12:51:12 I think there's actually a bug filed against the p
+}
+
void HostResolverMojo::CancelRequest(RequestHandle req) {
DCHECK(thread_checker_.CalledOnValidThread());
// Deleting the Job closes the HostResolverRequestClient connection,
« net/dns/host_resolver_impl_unittest.cc ('K') | « net/dns/host_resolver_mojo.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698