Implement RestartWithAuth for NewFtpTransaction.

net/ftp/ftp_response_info.h

 // Copyright (c) 2008 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_FTP_FTP_RESPONSE_INFO_H_
 #define NET_FTP_FTP_RESPONSE_INFO_H_
 
-#include "net/base/auth.h"
+#include "base/time.h"
 
 namespace net {
 
-class FtpResponseInfo {
+struct FtpResponseInfo {

[wtc, 2009/08/25 20:52:50]

Please change this back to 'class'.

We use 'class' deliberately so that when people write
forward declarations for the types in the net module,
they can just write 'class XXX', without having to look
up whether the type is a class or struct.

  public:
-  FtpResponseInfo() : is_directory_listing(false) {
+  FtpResponseInfo() : auth_needed(false), is_directory_listing(false) {
   }
 
-  // Non-null when authentication is required.
-  scoped_refptr<AuthChallengeInfo> auth_challenge;

[wtc, 2009/08/25 20:52:50]

Is a bool (auth_needed) all we need for FTP?
I guess AuthChallengeInfo is designed for HTTP auth, and
is an overkill for FTP?

+  // True if authentication failed and valid authentication credentials are
+  // needed.
+  bool auth_needed;

[wtc, 2009/08/25 20:52:50]

Nit: rename this member needs_auth to match the
NeedsAuth method of URLRequestJob.

 
   // The time at which the request was made that resulted in this response.
   // For cached responses, this time could be "far" in the past.
   base::Time request_time;
 
   // The time at which the response headers were received.  For cached
   // responses, this time could be "far" in the past.
   base::Time response_time;
 
   // True if the response data is of a directory listing.
   bool is_directory_listing;
 };
 
 }  // namespace net
 
 #endif  // NET_FTP_FTP_RESPONSE_INFO_H_