Propagate the remote socket address to URLRequest and to ViewHostMsg_FrameNavigate.

net/url_request/url_request_ftp_job.cc

Index: net/url_request/url_request_ftp_job.cc
diff --git a/net/url_request/url_request_ftp_job.cc b/net/url_request/url_request_ftp_job.cc
index 8c170ff3580002108259d1972363e18fb2e50c1d..9c77dea3facc3ac6fe0bc61ec030c15c94f5e068 100644
--- a/net/url_request/url_request_ftp_job.cc
+++ b/net/url_request/url_request_ftp_job.cc
@@ -52,6 +52,15 @@ bool URLRequestFtpJob::GetMimeType(std::string* mime_type) const {
   return false;
 }
 
+std::string URLRequestFtpJob::GetSocketAddress() const {
+  if (!transaction_.get()) {

[Paweł Hajdan Jr., 2011/02/11 19:34:54]

It seems that other methods here assume transaction_ is non-NULL. Is this
NULL-check really needed?

[Brian Ryner, 2011/02/11 23:01:07]

I was trying to make sure it was safe to call GetSocketAddress() at any time,
such as before Start() is called, or after Kill() is called.

+    return "";
+  }
+
+  const FtpResponseInfo* response_info = transaction_->GetResponseInfo();
+  return response_info ? response_info->socket_address : "";

[Paweł Hajdan Jr., 2011/02/11 19:34:54]

Similarly here, do we need to NULL-check?

[Brian Ryner, 2011/02/11 23:01:07]

Here, probably not, it looks like GetResponseInfo() can never return NULL. 
Fixed.

+}
+
 URLRequestFtpJob::~URLRequestFtpJob() {
 }