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

net/url_request/url_request_ftp_job.h

Index: net/url_request/url_request_ftp_job.h
diff --git a/net/url_request/url_request_ftp_job.h b/net/url_request/url_request_ftp_job.h
index 05cbb33979c8e6aad32b4f8e93259833a28ea36c..4a993bcb779e766f8c1692dc3d69c1606148ef77 100644
--- a/net/url_request/url_request_ftp_job.h
+++ b/net/url_request/url_request_ftp_job.h
@@ -17,13 +17,19 @@
 
 namespace net {
 
+class NetworkDelegate;
+class FtpTransactionFactory;
+class FtpAuthCache;
 class URLRequestContext;
 
 // 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);
 
   static URLRequestJob* Factory(URLRequest* request,
                                 const std::string& scheme);

[mmenke, 2012/06/26 18:36:10]

The plan is to get rid of this function in favor of the FtpProtocolHandler,
right?  If so, could you add a TODO along those lines?

[shalev, 2012/06/26 19:05:01]

Done.

@@ -69,6 +75,9 @@ class URLRequestFtpJob : public URLRequestJob {
 
   base::WeakPtrFactory<URLRequestFtpJob> weak_factory_;
 
+  FtpTransactionFactory* ftp_transaction_factory_;
+  FtpAuthCache* ftp_auth_cache_;
+
   DISALLOW_COPY_AND_ASSIGN(URLRequestFtpJob);
 };