Utility to resolve an hostname using Chromium's code in net/dns

net/base/host_resolver_impl.cc

Index: net/base/host_resolver_impl.cc
diff --git a/net/base/host_resolver_impl.cc b/net/base/host_resolver_impl.cc
index 541246f8b4d3785d4f644bc3c0830120b668f660..8089b6c81c605be29ddc3cd17e23291b6b667f32 100644
--- a/net/base/host_resolver_impl.cc
+++ b/net/base/host_resolver_impl.cc
@@ -434,6 +434,8 @@ class PriorityTracker {
   size_t counts_[NUM_PRIORITIES];
 };
 
+}  // anonymous namespace
+
 //-----------------------------------------------------------------------------
 
 HostResolver* CreateHostResolver(size_t max_concurrent_resolves,
@@ -459,10 +461,6 @@ HostResolver* CreateHostResolver(size_t max_concurrent_resolves,
   return resolver;
 }
 
-}  // anonymous namespace
-
-//-----------------------------------------------------------------------------
-
 HostResolver* CreateSystemHostResolver(size_t max_concurrent_resolves,
                                        size_t max_retry_attempts,
                                        NetLog* net_log) {
@@ -1070,6 +1068,7 @@ class HostResolverImpl::DnsTask {
         base::Bind(&DnsTask::OnTransactionComplete, base::Unretained(this),
                    base::TimeTicks::Now()),
         net_log_);
+

[szym, 2012/05/14 18:48:32]

No need for this empty line.

[Daniele, 2012/05/17 23:04:34]

Done.

     DCHECK(transaction_.get());
   }
 
@@ -1557,10 +1556,9 @@ HostResolverImpl::HostResolverImpl(
 #endif
 
   if (dns_config_service_.get()) {
-    dns_config_service_->Watch(
-        base::Bind(&HostResolverImpl::OnDnsConfigChanged,
-                   base::Unretained(this)));
     dns_client_ = DnsClient::CreateClient(net_log_);
+    dns_config_service_->Watch(base::Bind(&HostResolverImpl::OnDnsConfigChanged,

[szym, 2012/05/14 18:48:32]

Why did you change the order? I'm okay with the new order, but I'm wondering
about the reasoning here.

[Daniele, 2012/05/17 23:04:34]

I changed it because of the way I was calling the callback from
DnsConfigServiceMock inside the Watch method. I needed dns_client_ to be created
before calling the callback.
Now I changed the code as you suggested and there is no need for this.

On 2012/05/14 18:48:32, szym wrote:

+                                           base::Unretained(this)));
   }
 }