Split ProxyResolverV8Tracing into an implementation and a wrapper.

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 "base/strings/utf_string_conversions.h"
 #include "net/base/test_completion_callback.h"
 #include "net/proxy/mock_proxy_resolver.h"
 #include "net/proxy/proxy_resolver_error_observer.h"
@@ skipping 58 matching lines @@
             base::Unretained(this)),
         mojo::GetProxy(&factory_));
     mock_factory_owner_.reset(new TestProxyResolverFactory(&waiter_));
     mock_factory_ = mock_factory_owner_.get();
   }
 
   void OnConnectionError() override { waiter_.NotifyEvent(CONNECTION_ERROR); }
 
   scoped_ptr<ProxyResolverFactory> CreateFakeProxyResolverFactory(
       HostResolver* host_resolver,
-      scoped_ptr<ProxyResolverErrorObserver> error_observer,
-      const ProxyResolver::LoadStateChangedCallback& callback) {
+      scoped_ptr<ProxyResolverErrorObserver> error_observer) {
     EXPECT_TRUE(host_resolver);
-    EXPECT_FALSE(callback.is_null());
     DCHECK(mock_factory_owner_);
     return mock_factory_owner_.Pass();
   }
 
   void OnFakeProxyInstanceDestroyed() {
     instances_destroyed_++;
     waiter_.NotifyEvent(RESOLVER_DESTROYED);
   }
 
   void ReportResult(int32_t error) override { create_callback_.Run(error); }
@@ skipping 205 matching lines @@
   waiter_.WaitForEvent(RESOLVER_CREATED);
   EXPECT_EQ(0, instances_destroyed_);
   ASSERT_EQ(1u, mock_factory_->pending_requests().size());
   EXPECT_EQ(base::ASCIIToUTF16(kScriptData),
             mock_factory_->pending_requests()[0]->script_data()->utf16());
   factory_.reset();
   waiter_.WaitForEvent(CONNECTION_ERROR);
 }
 
 }  // namespace net