Removed static factories for data, ftp, file, and about 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_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
+#include "net/ftp/ftp_auth_cache.h"
 #include "net/url_request/url_request_job_factory.h"
 
 namespace net {
 
-class FtpAuthCache;
 class FtpTransactionFactory;
 class NetworkDelegate;
 class URLRequestJob;
 
 // Implements a ProtocolHandler for FTP.
 class NET_EXPORT FtpProtocolHandler :
     public URLRequestJobFactory::ProtocolHandler {
  public:
-  FtpProtocolHandler(FtpTransactionFactory* ftp_transaction_factory,
-                     FtpAuthCache* ftp_auth_cache);
+  explicit FtpProtocolHandler(FtpTransactionFactory* ftp_transaction_factory);
   virtual URLRequestJob* MaybeCreateJob(
       URLRequest* request, NetworkDelegate* network_delegate) const OVERRIDE;
 
  private:
+  friend class FtpTestURLRequestContext;
+
   FtpTransactionFactory* ftp_transaction_factory_;
-  FtpAuthCache* ftp_auth_cache_;
+  mutable FtpAuthCache ftp_auth_cache_;

[Paweł Hajdan Jr., 2013/04/29 18:45:26]

This seems wrong. If MaybeCreateJob is supposed to be const, this looks like
cheating.

Note that it's fine to use mutable for things _unrelated_ to the abstraction
like some bookkeeping/debugging/logging etc. However, FtpAuthCache is definitely
related to the core of what is going on here.

 
   DISALLOW_COPY_AND_ASSIGN(FtpProtocolHandler);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_FTP_PROTOCOL_HANDLER_H_