Reimplements net::AddressList without struct addrinfo.

chrome/browser/net/network_stats.h

Index: chrome/browser/net/network_stats.h
diff --git a/chrome/browser/net/network_stats.h b/chrome/browser/net/network_stats.h
index da6dd94502a0b7264b0bc1139ff4cf58ee6449d6..f125f666431defeb5203adc41e0757164f14ccb3 100644
--- a/chrome/browser/net/network_stats.h
+++ b/chrome/browser/net/network_stats.h
@@ -151,7 +151,7 @@ class NetworkStats {
   base::TimeTicks start_time() const { return start_time_; }
 
   // Returns |addresses_|.
-  net::AddressList GetAddressList() const { return addresses_; }
+  const net::AddressList& get_address_list() const { return addresses_; }

[eroman, 2012/05/04 19:37:36]

Why not call this addresses()?

our hacker style for accessors is to name it the same as the private variable
(without the trailing underscore). For setters you prefix with "set_". See for
instance the other accessors in this header: "load_size(), "start_time".

[szym, 2012/05/04 20:20:21]

Of course! Wasn't thinking.

 
   // Returns packets_received_mask_ (used by unit tests).
   uint32 packets_received_mask() const { return packets_received_mask_; }