Implement RestartWithAuth for NewFtpTransaction.

net/ftp/ftp_response_info.h

Index: net/ftp/ftp_response_info.h
diff --git a/net/ftp/ftp_response_info.h b/net/ftp/ftp_response_info.h
index b3c3361dddc3703315a7ed10d7e738e5ea121a8c..3361120bca0a354f2dfa011b82a675be22ecb3b6 100644
--- a/net/ftp/ftp_response_info.h
+++ b/net/ftp/ftp_response_info.h
@@ -5,17 +5,18 @@
 #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.