Change ProxyResolver::GetProxyForURL() to take a scoped_ptr<Request>* rather than a RequestHandle*

net/proxy/proxy_resolver_v8_tracing.h

Index: net/proxy/proxy_resolver_v8_tracing.h
diff --git a/net/proxy/proxy_resolver_v8_tracing.h b/net/proxy/proxy_resolver_v8_tracing.h
index f5f66cf7486797d0c6282f7dab361b2942cfa283..d8b974b7fc2a2c00cacab1cc0ca2da26303dc93e 100644
--- a/net/proxy/proxy_resolver_v8_tracing.h
+++ b/net/proxy/proxy_resolver_v8_tracing.h
@@ -19,6 +19,7 @@ class HostResolver;
 // ProxyResolverV8Tracing is a non-blocking proxy resolver.
 class NET_EXPORT ProxyResolverV8Tracing {
  public:
+  class Request : public ProxyResolver::Request {};

[eroman, 2016/02/24 03:08:07]

Why is this needed? Can it just be declared in the .cc file?

   // Bindings is an interface used by ProxyResolverV8Tracing to delegate
   // per-request functionality. Each instance will be destroyed on the origin
   // thread of the ProxyResolverV8Tracing when the request completes or is
@@ -55,15 +56,8 @@ class NET_EXPORT ProxyResolverV8Tracing {
   virtual void GetProxyForURL(const GURL& url,
                               ProxyInfo* results,
                               const CompletionCallback& callback,
-                              ProxyResolver::RequestHandle* request,
+                              scoped_ptr<ProxyResolver::Request>* request,
                               scoped_ptr<Bindings> bindings) = 0;
-
-  // Cancels |request|.
-  virtual void CancelRequest(ProxyResolver::RequestHandle request) = 0;
-
-  // Gets the LoadState for |request|.
-  virtual LoadState GetLoadState(
-      ProxyResolver::RequestHandle request) const = 0;
 };
 
 // A factory for ProxyResolverV8Tracing instances. The default implementation,