Refactoring: ProtocolHandler::MaybeCreateJob takes NetworkDelegate as argument

net/url_request/url_request_about_job.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.
 
 // Simple implementation of about: protocol handler that treats everything as
 // about:blank.  No other about: features should be available to web content,
 // so they're not implemented here.
 
 #include "net/url_request/url_request_about_job.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_context.h"

[mmenke, 2012/08/21 15:28:49]

Don't think we need either of these two.

[shalev, 2012/08/22 20:34:10]

Done.

 
 namespace net {
 
-URLRequestAboutJob::URLRequestAboutJob(URLRequest* request)
-    : URLRequestJob(request, request->context()->network_delegate()),
+URLRequestAboutJob::URLRequestAboutJob(URLRequest* request,
+                                       NetworkDelegate* network_delegate)
+    : URLRequestJob(request, network_delegate),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) {
 }
 
 // static
 URLRequestJob* URLRequestAboutJob::Factory(URLRequest* request,
+                                           NetworkDelegate* network_delegate,
                                            const std::string& scheme) {
-  return new URLRequestAboutJob(request);
+  return new URLRequestAboutJob(request, network_delegate);
 }
 
 void URLRequestAboutJob::Start() {
   // Start reading asynchronously so that all error reporting and data
   // callbacks happen as they would for network requests.
   MessageLoop::current()->PostTask(
       FROM_HERE,
       base::Bind(&URLRequestAboutJob::StartAsync, weak_factory_.GetWeakPtr()));
 }
 
 bool URLRequestAboutJob::GetMimeType(std::string* mime_type) const {
   *mime_type = "text/html";
   return true;
 }
 
 URLRequestAboutJob::~URLRequestAboutJob() {
 }
 
 void URLRequestAboutJob::StartAsync() {
   NotifyHeadersComplete();
 }
 
 }  // namespace net