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

Unified Diff: net/dns/mapped_host_resolver.cc

Issue 2116983002: Change HostResolver::Resolve() to take an std::unique_ptr<Request>* rather than a RequestHandle* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: http_stream_factory_impl_job_controller_unittest RequestHandle* to unique_ptr Created 4 years, 5 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/dns/mapped_host_resolver.cc
diff --git a/net/dns/mapped_host_resolver.cc b/net/dns/mapped_host_resolver.cc
index e102d502444227db01876941a3f093da764ad102..43ad71acdde54a566a41ac067214ba9d4d765ecd 100644
--- a/net/dns/mapped_host_resolver.cc
+++ b/net/dns/mapped_host_resolver.cc
@@ -23,14 +23,14 @@ int MappedHostResolver::Resolve(const RequestInfo& original_info,
RequestPriority priority,
AddressList* addresses,
const CompletionCallback& callback,
- RequestHandle* out_req,
+ std::unique_ptr<Request>* request,
const BoundNetLog& net_log) {
RequestInfo info = original_info;
int rv = ApplyRules(&info);
if (rv != OK)
return rv;
- return impl_->Resolve(info, priority, addresses, callback, out_req, net_log);
+ return impl_->Resolve(info, priority, addresses, callback, request, net_log);
}
int MappedHostResolver::ResolveFromCache(const RequestInfo& original_info,
@@ -44,10 +44,6 @@ int MappedHostResolver::ResolveFromCache(const RequestInfo& original_info,
return impl_->ResolveFromCache(info, addresses, net_log);
}
-void MappedHostResolver::CancelRequest(RequestHandle req) {
- impl_->CancelRequest(req);
-}
-
void MappedHostResolver::SetDnsClientEnabled(bool enabled) {
impl_->SetDnsClientEnabled(enabled);
}

Powered by Google App Engine
This is Rietveld 408576698