Refactoring: ProtocolHandler::MaybeCreateJob takes NetworkDelegate as argument

chrome_frame/test/net/test_automation_provider.cc

 // Copyright (c) 2012 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 "chrome_frame/test/net/test_automation_provider.h"
 
 #include "base/command_line.h"
 #include "base/file_version_info.h"
 #include "base/path_service.h"
 #include "chrome/common/automation_messages.h"
@@ skipping 51 matching lines @@
     DCHECK(tab_handle_ == -1) << "Currently only support one tab";
     tab_handle_ = msg->routing_id();
     DVLOG(1) << "Got tab handle: " << tab_handle_;
     DCHECK(tab_handle_ != -1 && tab_handle_ != 0);
     delegate_->OnInitialTabLoaded();
   }
 
   return AutomationProvider::Send(msg);
 }
 
-net::URLRequestJob* TestAutomationProvider::Factory(net::URLRequest* request,
-                                                    const std::string& scheme) {
+net::URLRequestJob* TestAutomationProvider::Factory(
+    net::URLRequest* request,
+    net::NetworkDelegate* network_delegate,
+    const std::string& scheme) {
   if (CFTestsDisabled())
     return NULL;
 
   if (request->url().SchemeIs("http") || request->url().SchemeIs("https")) {
     // Only look at requests that don't have any user data.
     // ResourceDispatcherHost uses the user data for requests that it manages.
     // We don't want to mess with those.
 
     // We could also check if the current thread is our TestUrlRequest thread
     // and only intercept requests that belong to that thread.
     if (g_provider_instance_ && request->GetUserData(NULL) == NULL &&
         g_provider_instance_->tab_handle_ != -1) {
       // We generate our own request id which is also what
       // ResourceDispatcherHost does (well, the id is actually generated by
       // ResourceDispatcher).  Since these requests are divided into with
       // and without userdata, we're OK.  However, just to make debugging
       // a little easier, we have a significantly higher start value.
       static int new_id = 0x00100000;
-      URLRequestAutomationJob* job = new URLRequestAutomationJob(request,
+      URLRequestAutomationJob* job = new URLRequestAutomationJob(
+          request, network_delegate,
           g_provider_instance_->tab_handle_, new_id++,
           g_provider_instance_->automation_resource_message_filter_, false);
       return job;
     }
   }
 
   return NULL;
 }
 
 std::string TestAutomationProvider::GetProtocolVersion() {
@@ skipping 20 matching lines @@
 
 // static
 TestAutomationProvider* TestAutomationProvider::NewAutomationProvider(
     Profile* p, const std::string& channel,
     TestAutomationProviderDelegate* delegate) {
   TestAutomationProvider* automation = new TestAutomationProvider(p, delegate);
   automation->InitializeChannel(channel);
   automation->SetExpectedTabCount(1);
   return automation;
 }