Revert of Change ProxyResolver::GetProxyForURL() to take a scoped_ptr<Request>* rather than a Reque…

net/proxy/mojo_proxy_resolver_factory_impl_unittest.cc

 // Copyright 2015 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 "net/proxy/mojo_proxy_resolver_factory_impl.h"
 
 #include <utility>
 
 #include "base/strings/utf_string_conversions.h"
 #include "mojo/public/cpp/bindings/binding.h"
@@ skipping 13 matching lines @@
   explicit FakeProxyResolver(const base::Closure& on_destruction)
       : on_destruction_(on_destruction) {}
 
   ~FakeProxyResolver() override { on_destruction_.Run(); }
 
  private:
   // ProxyResolverV8Tracing overrides.
   void GetProxyForURL(const GURL& url,
                       ProxyInfo* results,
                       const CompletionCallback& callback,
-                      scoped_ptr<ProxyResolver::Request>* request,
+                      ProxyResolver::RequestHandle* request,
                       scoped_ptr<Bindings> bindings) override {}
 
+  void CancelRequest(ProxyResolver::RequestHandle request) override {}
+
+  LoadState GetLoadState(ProxyResolver::RequestHandle request) const override {
+    return LOAD_STATE_RESOLVING_PROXY_FOR_URL;
+  }
+
   const base::Closure on_destruction_;
 };
 
 enum Event {
   NONE,
   RESOLVER_CREATED,
   CONNECTION_ERROR,
   RESOLVER_DESTROYED,
 };
 
@@ skipping 160 matching lines @@
       base::Bind(&MojoProxyResolverFactoryImplTest::OnConnectionError,
                  base::Unretained(this)));
   waiter_.WaitForEvent(RESOLVER_CREATED);
   EXPECT_EQ(0, instances_destroyed_);
   ASSERT_EQ(1u, mock_factory_->requests_handled());
   factory_.reset();
   waiter_.WaitForEvent(CONNECTION_ERROR);
 }
 
 }  // namespace net