Reimplements net::AddressList without struct addrinfo.

net/base/host_resolver_proc.h

Index: net/base/host_resolver_proc.h
diff --git a/net/base/host_resolver_proc.h b/net/base/host_resolver_proc.h
index 0acb31eeeb2b4139eb6551c85047a8f81a3495b7..c55c96830b77ff595e09d3883e8b33b8c51e36d7 100644
--- a/net/base/host_resolver_proc.h
+++ b/net/base/host_resolver_proc.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 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.
 
@@ -16,6 +16,12 @@ namespace net {
 
 class AddressList;
 
+// Sets the port on each element in |list| to |port|.
+void NET_EXPORT SetPortOnAddressList(uint16 port, AddressList* list);

[eroman, 2012/05/04 01:08:41]

This doesn't feel like the right place to house this code.

Maybe address_list.h would be a better fit for this first function. The second
function is a little bit too specific on how the lists were constructed, so
doesn't fit as cleanly into address_list.h

This is probably more appropriate in address_list.h, or part of
host_resolver_impl.cc which is where it is used. (who else uses this?)

I dunno; net_util.h ends up being our dumping ground for these sorts of things.

[szym, 2012/05/04 02:38:29]

Only HostResolver-related code uses this, but this includes
mock_host_resolver.cc
On the other hand, there is little benefit from such optimization in
MockHostResolver, so I'll move EnsurePort... to host_resolver_impl.cc and
SetPort... to address_list.cc

[eroman, 2012/05/04 04:20:22]

Sounds like a good plan.

+
+// As above, but skips work if first port on |list| matches |port|.
+void NET_EXPORT EnsurePortOnAddressList(uint16 port, AddressList* list);
+
 // Interface for a getaddrinfo()-like procedure. This is used by unit-tests
 // to control the underlying resolutions in HostResolverImpl. HostResolverProcs
 // can be chained together; they fallback to the next procedure in the chain