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

net/url_request/ftp_protocol_handler.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_FTP_PROTOCOL_HANDLER_H_
+#define NET_URL_REQUEST_FTP_PROTOCOL_HANDLER_H_
+#pragma once
+
+#include "net/url_request/url_request_job_factory.h"

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

include "base/basictypes.h" for DISALLOW_COPY_AND_ASSIGN, and
"base/compiler_specific.h" for OVERRIDE.

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

Done.

+
+namespace net {
+
+class URLRequestJob;
+class NetworkDelegate;
+class FtpTransactionFactory;
+class FtpAuthCache;

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

nit:  Alphabetize.

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

Done.

+
+// Implements a ProtocolHandler for FTP.
+class NET_EXPORT FtpProtocolHandler :
+    public URLRequestJobFactory::ProtocolHandler {
+ public:
+  FtpProtocolHandler(NetworkDelegate* network_delegate,
+                     FtpTransactionFactory* ftp_transaction_factory,
+                     FtpAuthCache* ftp_auth_cache);
+  virtual URLRequestJob* MaybeCreateJob(URLRequest* request) const OVERRIDE;
+
+ private:
+  NetworkDelegate* network_delegate_;
+  FtpTransactionFactory* ftp_transaction_factory_;
+  FtpAuthCache* ftp_auth_cache_;
+  DISALLOW_COPY_AND_ASSIGN(FtpProtocolHandler);

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

nit:  Add a line break just above DISALLOW_COPY_AND_ASSIGN, to separate it from
the variables.

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

Done.

+};
+
+}  // namespace net
+
+#endif  // NET_URL_REQUEST_FTP_PROTOCOL_HANDLER_H_