Second patch in making destructors of refcounted objects private.

net/base/host_resolver_impl.cc

 // Copyright (c) 2006-2008 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.
 
 #include "net/base/host_resolver_impl.h"
 
 #if defined(OS_WIN)
 #include <ws2tcpip.h>
 #include <wspiapi.h>  // Needed for Win2k compat.
 #elif defined(OS_POSIX)
@@ skipping 131 matching lines @@
     : public base::RefCountedThreadSafe<HostResolverImpl::Job> {
  public:
   Job(HostResolverImpl* resolver, const Key& key)
       : key_(key),
         resolver_(resolver),
         origin_loop_(MessageLoop::current()),
         resolver_proc_(resolver->effective_resolver_proc()),
         error_(OK) {
   }
 
-  ~Job() {
-    // Free the requests attached to this job.
-    STLDeleteElements(&requests_);
-  }
-
   // Attaches a request to this job. The job takes ownership of |req| and will
   // take care to delete it.
   void AddRequest(Request* req) {
     req->set_job(this);
     requests_.push_back(req);
   }
 
   // Called from origin loop.
   void Start() {
     // Dispatch the job to a worker thread.
@@ skipping 41 matching lines @@
   const Key& key() const {
     return key_;
   }
 
   // Called from origin thread.
   const RequestsList& requests() const {
     return requests_;
   }
 
  private:
+  friend class base::RefCountedThreadSafe<HostResolverImpl::Job>;
+
+  ~Job() {
+    // Free the requests attached to this job.
+    STLDeleteElements(&requests_);
+  }
+
   void DoLookup() {
     // Running on the worker thread
     error_ = ResolveAddrInfo(resolver_proc_,
                              key_.hostname,
                              key_.address_family,
                              &results_);
 
     Task* reply = NewRunnableMethod(this, &Job::OnLookupComplete);
 
     // The origin loop could go away while we are trying to post to it, so we
@@ skipping 324 matching lines @@
     }
 
     LoadLog::EndEvent(
         load_log, LoadLog::TYPE_HOST_RESOLVER_IMPL_OBSERVER_ONCANCEL);
   }
 
   LoadLog::EndEvent(load_log, LoadLog::TYPE_HOST_RESOLVER_IMPL);
 }
 
 }  // namespace net