Change HostResolver::Resolve() to take an std::unique_ptr<Request>* rather than a RequestHandle*

net/dns/host_resolver_impl_fuzzer.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <memory>
 #include <vector>
 
@@ skipping 20 matching lines @@
 };
 
 class DnsRequest {
  public:
   DnsRequest(net::HostResolver* host_resolver,
              net::FuzzedDataProvider* data_provider,
              std::vector<std::unique_ptr<DnsRequest>>* dns_requests)
       : host_resolver_(host_resolver),
         data_provider_(data_provider),
         dns_requests_(dns_requests),
-        handle_(nullptr),
+        request_(),

[mmenke, 2016/07/19 19:03:56]

this isn't needed.

[maksims (do not use this acc), 2016/07/21 07:12:45]

Done.

         is_running_(false) {}
 
   ~DnsRequest() {
     if (is_running_)
       Cancel();

[mmenke, 2016/07/19 19:03:56]

This is no longer needed - deleting this DnsRequest will delete request_, which
will cancel the request.  Which is one of the nice things about the new API.

[maksims (do not use this acc), 2016/07/21 07:12:45]

Done.

   }
 
   // Creates and starts a DNS request using fuzzed parameters. If the request
   // doesn't complete synchronously, adds it to |dns_requests|.
   static void CreateRequest(
       net::HostResolver* host_resolver,
       net::FuzzedDataProvider* data_provider,
       std::vector<std::unique_ptr<DnsRequest>>* dns_requests) {
     std::unique_ptr<DnsRequest> dns_request(
         new DnsRequest(host_resolver, data_provider, dns_requests));
@@ skipping 91 matching lines @@
 
     // Decide if should be a cache-only resolution.
     if (data_provider_->ConsumeBool()) {
       return host_resolver_->ResolveFromCache(info, &address_list_,
                                               net::BoundNetLog());
     }
 
     info.set_allow_cached_response(data_provider_->ConsumeBool());
     return host_resolver_->Resolve(
         info, priority, &address_list_,
-        base::Bind(&DnsRequest::OnCallback, base::Unretained(this)), &handle_,
+        base::Bind(&DnsRequest::OnCallback, base::Unretained(this)), &request_,
         net::BoundNetLog());
   }
 
   // Waits until the request is done, if it isn't done already.
   void WaitUntilDone() {
     CHECK(!run_loop_);
     if (is_running_) {
       run_loop_.reset(new base::RunLoop());
       run_loop_->Run();
       run_loop_.reset();
       CHECK(!is_running_);

[mmenke, 2016/07/19 19:03:56]

I'd suggest getting rid of is_running_, and DCHECK on request_ here instead. 
Also need to reset request_ in OnCallback, where is_running_ is set to false.

[maksims (do not use this acc), 2016/07/21 07:12:45]

Done.

     }
   }
 
   // Cancel the request, if not already completed. Otherwise, does nothing.
   void Cancel() {
     if (is_running_)
-      host_resolver_->CancelRequest(handle_);
+      request_.reset();

[mmenke, 2016/07/19 19:03:56]

nit:  Can unconditionally reset the request here.

[maksims (do not use this acc), 2016/07/21 07:12:45]

Done.

     is_running_ = false;
   }
 
   net::HostResolver* host_resolver_;
   net::FuzzedDataProvider* data_provider_;
   std::vector<std::unique_ptr<DnsRequest>>* dns_requests_;
 
-  net::HostResolver::RequestHandle handle_;
+  std::unique_ptr<net::HostResolver::Request> request_;
   net::AddressList address_list_;
 
   bool is_running_;
 
   std::unique_ptr<base::RunLoop> run_loop_;
 
   DISALLOW_COPY_AND_ASSIGN(DnsRequest);
 };
 
 }  // namespace
@@ skipping 35 matching lines @@
           break;
       }
     }
   }
 
   // Clean up any pending tasks, after deleting everything.
   base::RunLoop().RunUntilIdle();
 
   return 0;
 }