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..fe40a205731d78ee41f7093248b64c615c09dc51 100644
--- a/net/url_request/url_request_ftp_job.h
+++ b/net/url_request/url_request_ftp_job.h
@@ -15,6 +15,11 @@
 #include "net/ftp/ftp_transaction.h"
 #include "net/url_request/url_request_job.h"
 

[Paweł Hajdan Jr., 2012/06/07 19:30:21]

nit: Why an empty line? Keep things sorted and forward-declare where possible as
Erik said.

[shalev, 2012/06/21 20:04:55]

Done.

+#include "net/url_request/url_request_job_factory.h"
+#include "net/base/network_delegate.h"

[erikwright (departed), 2012/06/07 19:14:51]

presumably these includes are no longer required here? Just some forward-decls?

[shalev, 2012/06/21 20:04:55]

Done.

+#include "net/ftp/ftp_transaction_factory.h"
+#include "net/ftp/ftp_auth_cache.h"
+
 namespace net {
 
 class URLRequestContext;
@@ -23,7 +28,10 @@ class URLRequestContext;
 // 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);
@@ -31,6 +39,11 @@ class URLRequestFtpJob : public URLRequestJob {
   // Overridden from URLRequestJob:
   virtual bool GetMimeType(std::string* mime_type) const OVERRIDE;
   virtual HostPortPair GetSocketAddress() const OVERRIDE;
+  NetworkDelegate* network_delegate() const { return network_delegate_; }
+  FtpTransactionFactory* ftp_transaction_factory() {

[erikwright (departed), 2012/06/07 19:14:51]

Why are these accessors added?

[Paweł Hajdan Jr., 2012/06/07 19:30:21]

Yup, they really shouldn't be needed.

[shalev, 2012/06/21 20:04:55]

Done.

+    return ftp_transaction_factory_;
+  }
+  FtpAuthCache* ftp_auth_cache() { return ftp_auth_cache_; }
 
  private:
   virtual ~URLRequestFtpJob();
@@ -70,6 +83,10 @@ class URLRequestFtpJob : public URLRequestJob {
   base::WeakPtrFactory<URLRequestFtpJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequestFtpJob);

[erikwright (departed), 2012/06/07 19:14:51]

DISALLOW_... should be the last line in the private member section.

[shalev, 2012/06/21 20:04:55]

Done.

+
+  NetworkDelegate* network_delegate_;
+  FtpTransactionFactory* ftp_transaction_factory_;
+  FtpAuthCache* ftp_auth_cache_;
 };
 
 }  // namespace net