Propagate the remote socket address to URLRequest and to ViewHostMsg_FrameNavigate.

net/base/host_port_pair.h

 // Copyright (c) 2010 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_BASE_HOST_PORT_PAIR_H_
 #define NET_BASE_HOST_PORT_PAIR_H_
 #pragma once
 
 #include <string>
 #include "base/basictypes.h"
 
+struct addrinfo;
 class GURL;
 
 namespace net {
 
 class HostPortPair {
  public:
   HostPortPair();
   // If |in_host| represents an IPv6 address, it should not bracket the address.
   HostPortPair(const std::string& in_host, uint16 in_port);
 
   // Creates a HostPortPair for the origin of |url|.
   static HostPortPair FromURL(const GURL& url);
 
+  // Creates a HostPortPair from an addrinfo struct.
+  static HostPortPair FromAddrInfo(const struct addrinfo* ai);

[eroman, 2011/02/18 02:48:30]

You might consider having this operate on the head of a |const AddressList&|
instead, since all the consumers are giving those. (Either approach seems
reasonable be to me, so whatever you prefer).

[Brian Ryner, 2011/02/18 04:41:59]

I think I'll stick with addrinfo, since it's a little more flexible.

On 2011/02/18 02:48:30, eroman wrote:

+
   // TODO(willchan): Define a functor instead.
   // Comparator function so this can be placed in a std::map.
   bool operator<(const HostPortPair& other) const {
     if (port_ != other.port_)
       return port_ < other.port_;
     return host_ < other.host_;
   }
 
   // Equality test of contents. (Probably another violation of style guide).
   bool Equals(const HostPortPair& other) const {
@@ skipping 26 matching lines @@
  private:
   // If |host_| represents an IPv6 address, this string will not contain
   // brackets around the address.
   std::string host_;
   uint16 port_;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_HOST_PORT_PAIR_H_