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

net/url_request/ftp_protocol_handler.cc

Index: net/url_request/ftp_protocol_handler.cc
diff --git a/net/url_request/ftp_protocol_handler.cc b/net/url_request/ftp_protocol_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a2b1a220dc18c84ab9b4b2de13a0faedda591521
--- /dev/null
+++ b/net/url_request/ftp_protocol_handler.cc
@@ -0,0 +1,34 @@
+// 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.
+
+#include "net/url_request/ftp_protocol_handler.h"
+#include "net/url_request/url_request_ftp_job.h"

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

Include "base/logging.h" for the DCHECKs.

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

Done.

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

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

These aren't needed, since they're in the header.

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

Done.

+
+FtpProtocolHandler::FtpProtocolHandler(
+    NetworkDelegate* network_delegate,
+    FtpTransactionFactory* ftp_transaction_factory,
+    FtpAuthCache* ftp_auth_cache)
+    : network_delegate_(network_delegate),
+      ftp_transaction_factory_(ftp_transaction_factory),
+      ftp_auth_cache_(ftp_auth_cache) {
+  DCHECK(ftp_transaction_factory_);
+  DCHECK(ftp_auth_cache_);
+}
+
+URLRequestJob* FtpProtocolHandler::MaybeCreateJob(
+    URLRequest* request) const {
+  return new URLRequestFtpJob(request,
+                              network_delegate_,
+                              ftp_transaction_factory_,
+                              ftp_auth_cache_);
+}
+
+}  // namespace net