Replaced static URLRequestFtpJob factory with non-static protocol handler for FTP jobs.

net/url_request/url_request_ftp_job.h

 // 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.
 
 #ifndef NET_URL_REQUEST_URL_REQUEST_FTP_JOB_H_
 #define NET_URL_REQUEST_URL_REQUEST_FTP_JOB_H_
 #pragma once
 
 #include <string>
 
 #include "base/memory/weak_ptr.h"
 #include "net/base/auth.h"
 #include "net/base/completion_callback.h"
 #include "net/ftp/ftp_request_info.h"
 #include "net/ftp/ftp_transaction.h"
 #include "net/url_request/url_request_job.h"
 
 namespace net {
 
+class NetworkDelegate;
+class FtpTransactionFactory;
+class FtpAuthCache;
 class URLRequestContext;

[mmenke, 2012/06/27 17:20:07]

While you're here, can you remove the  URLRequestContext forward declaration? 
It's not used in this file.

[shalev, 2012/06/27 20:10:43]

Done.

 
 // A URLRequestJob subclass that is built on top of FtpTransaction. It
 // provides an implementation for FTP.
 class URLRequestFtpJob : public URLRequestJob {
  public:
-  explicit URLRequestFtpJob(URLRequest* request);
+  URLRequestFtpJob(URLRequest* request,
+                   NetworkDelegate* network_delegate,
+                   FtpTransactionFactory* ftp_transaction_factory,
+                   FtpAuthCache* ftp_auth_cache);
 
+  // TODO(shalev): get rid of this function in favor of FtpProtocolHandler.
   static URLRequestJob* Factory(URLRequest* request,
                                 const std::string& scheme);
 
   // Overridden from URLRequestJob:
   virtual bool GetMimeType(std::string* mime_type) const OVERRIDE;
   virtual HostPortPair GetSocketAddress() const OVERRIDE;
 
  private:
   virtual ~URLRequestFtpJob();
 
@@ skipping 24 matching lines @@
 
   FtpRequestInfo request_info_;
   scoped_ptr<FtpTransaction> transaction_;
 
   bool read_in_progress_;
 
   scoped_refptr<AuthData> server_auth_;
 
   base::WeakPtrFactory<URLRequestFtpJob> weak_factory_;
 
+  FtpTransactionFactory* ftp_transaction_factory_;
+  FtpAuthCache* ftp_auth_cache_;
+
   DISALLOW_COPY_AND_ASSIGN(URLRequestFtpJob);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_FTP_JOB_H_