[net] Hide SupportsWeakPtr from HostResolverImpl

Following recommendation by willchan@.


Review URL: https://chromiumcodereview.appspot.com/10947014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157837 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/net/base/host_resolver_impl.cc b/net/base/host_resolver_impl.cc
index 1f8635b..d9e1aa2 100644
--- a/net/base/host_resolver_impl.cc
+++ b/net/base/host_resolver_impl.cc
@@ -1150,7 +1150,8 @@
       // Sort could complete synchronously.
       client_->GetAddressSorter()->Sort(
           addr_list,
-          base::Bind(&DnsTask::OnSortComplete, AsWeakPtr(),
+          base::Bind(&DnsTask::OnSortComplete,
+                     AsWeakPtr(),
                      base::TimeTicks::Now(),
                      ttl));
     } else {
@@ -1222,7 +1223,7 @@
       const Key& key,
       RequestPriority priority,
       const BoundNetLog& request_net_log)
-      : resolver_(resolver->AsWeakPtr()),
+      : resolver_(resolver->weak_ptr_factory_.GetWeakPtr()),
         key_(key),
         priority_tracker_(priority),
         had_non_speculative_request_(false),
@@ -1670,6 +1671,7 @@
       max_queued_jobs_(job_limits.total_jobs * 100u),
       proc_params_(proc_params),
       default_address_family_(ADDRESS_FAMILY_UNSPECIFIED),
+      weak_ptr_factory_(this),
       dns_client_(dns_client.Pass()),
       received_dns_config_(false),
       ipv6_probe_monitoring_(false),
@@ -2064,7 +2066,7 @@
   DCHECK_EQ(dispatcher_.num_running_jobs(), jobs_to_abort.size());
 
   // Life check to bail once |this| is deleted.
-  base::WeakPtr<HostResolverImpl> self = AsWeakPtr();
+  base::WeakPtr<HostResolverImpl> self = weak_ptr_factory_.GetWeakPtr();
 
   // Then Abort them.
   for (size_t i = 0; self && i < jobs_to_abort.size(); ++i) {
@@ -2081,7 +2083,7 @@
   // http://crbug.com/117655
 
   // Life check to bail once |this| is deleted.
-  base::WeakPtr<HostResolverImpl> self = AsWeakPtr();
+  base::WeakPtr<HostResolverImpl> self = weak_ptr_factory_.GetWeakPtr();
 
   for (JobMap::iterator it = jobs_.begin(); self && it != jobs_.end(); ) {
     Job* job = it->second;
@@ -2123,7 +2125,7 @@
   received_dns_config_ = dns_config.IsValid();
 
   // Life check to bail once |this| is deleted.
-  base::WeakPtr<HostResolverImpl> self = AsWeakPtr();
+  base::WeakPtr<HostResolverImpl> self = weak_ptr_factory_.GetWeakPtr();
 
   // We want a new DnsSession in place, before we Abort running Jobs, so that
   // the newly started jobs use the new config.