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..c25eb3db21e59844252a001a13bcf4f40caf1971
--- /dev/null
+++ b/net/url_request/ftp_protocol_handler.cc
@@ -0,0 +1,32 @@
+// 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"
+
+namespace net {
+
+class NetworkDelegate;
+class FtpTransactionFactory;
+class FtpAuthCache;
+class URLRequest;
+
+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) {
+}

[willchan no longer on Chromium, 2012/06/11 21:02:27]

I suspect we should add DCHECKs here, since it'll just crash later if some of
these are NULL, right?

[erikwright (departed), 2012/06/11 21:05:40]

Yes.

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

Done.

+
+URLRequestJob* FtpProtocolHandler::MaybeCreateJob(
+    URLRequest* request) const OVERRIDE {
+  return new URLRequestFtpJob(request,
+                              network_delegate_,
+                              ftp_transaction_factory_,
+                              ftp_auth_cache_);
+}
+
+}  // namespace net