[net] HostResolverImpl + DnsTransaction + DnsConfigService = Asynchronous DNS ready for experiments.

net/dns/async_host_resolver.cc

Index: net/dns/async_host_resolver.cc
diff --git a/net/dns/async_host_resolver.cc b/net/dns/async_host_resolver.cc
index 72e5472419daf4df4983a4a4c60b7b80b1350d6b..5ce1685c6ab9503546bca8121d95dfedfc434886 100644
--- a/net/dns/async_host_resolver.cc
+++ b/net/dns/async_host_resolver.cc
@@ -347,7 +347,6 @@ void AsyncHostResolver::OnDnsTransactionComplete(
       std::make_pair(transaction->GetHostname(), transaction->GetType()));
   DCHECK(rit != requestlist_map_.end());
   RequestList& requests = rit->second;
-  int port = requests.empty() ? 80 : requests.front()->info().port();
 
   // Extract AddressList and TTL out of DnsResponse.
   AddressList addr_list;
@@ -367,11 +366,16 @@ void AsyncHostResolver::OnDnsTransactionComplete(
       }
     }
     if (!ip_addresses.empty())
-      addr_list = AddressList::CreateFromIPAddressList(ip_addresses, port);
+      addr_list = AddressList::CreateFromIPAddressList(ip_addresses, "");
     else
       result = ERR_NAME_NOT_RESOLVED;
   }
 
+  // NOTE: This is here only to compile, but async_host_resolver* is going

[cbentzel, 2012/02/10 19:51:08]

Is this note still correct?

[szym, 2012/02/10 21:49:36]

I haven't removed async_host_resolver* yet, but that would be part of this CL. 

The tests that are using the data in dns_test_util.h will be moved to
dns_response_unittest.cc

+  // away in this CL.
+  int port = requests.empty() ? 80 : requests.front()->info().port();
+  addr_list.SetPort(port);
+
   // Run callback of every request that was depending on this DNS request,
   // also notify observers.
   for (RequestList::iterator it = requests.begin(); it != requests.end(); ++it)