Second patch in making destructors of refcounted objects private.

net/base/host_resolver_impl.h

 // 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.
 
 #ifndef NET_BASE_HOST_RESOLVER_IMPL_H_
 #define NET_BASE_HOST_RESOLVER_IMPL_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 33 matching lines @@
   // Creates a HostResolver that caches up to |max_cache_entries| for
   // |cache_duration_ms| milliseconds. |resolver_proc| is used to perform
   // the actual resolves; it must be thread-safe since it is run from
   // multiple worker threads. If |resolver_proc| is NULL then the default
   // host resolver procedure is used (which is SystemHostResolverProc except
   // if overridden)
   HostResolverImpl(HostResolverProc* resolver_proc,
                    int max_cache_entries,
                    int cache_duration_ms);
 
-  // If any completion callbacks are pending when the resolver is destroyed,
-  // the host resolutions are cancelled, and the completion callbacks will not
-  // be called.
-  virtual ~HostResolverImpl();
-
   // HostResolver methods:
   virtual int Resolve(const RequestInfo& info,
                       AddressList* addresses,
                       CompletionCallback* callback,
                       RequestHandle* out_req,
                       LoadLog* load_log);
   virtual void CancelRequest(RequestHandle req);
   virtual void AddObserver(Observer* observer);
   virtual void RemoveObserver(Observer* observer);
   virtual HostCache* GetHostCache();
 
   // TODO(eroman): temp hack for http://crbug.com/15513
   virtual void Shutdown();
 
   virtual void SetDefaultAddressFamily(AddressFamily address_family) {
     default_address_family_ = address_family;
   }
 
  private:
   class Job;
   class Request;
   typedef std::vector<Request*> RequestsList;
   typedef HostCache::Key Key;
   typedef std::map<Key, scoped_refptr<Job> > JobMap;
   typedef std::vector<Observer*> ObserversList;
 
+  // If any completion callbacks are pending when the resolver is destroyed,
+  // the host resolutions are cancelled, and the completion callbacks will not
+  // be called.
+  virtual ~HostResolverImpl();
+
   // Returns the HostResolverProc to use for this instance.
   HostResolverProc* effective_resolver_proc() const {
     return resolver_proc_ ?
         resolver_proc_.get() : HostResolverProc::GetDefault();
   }
 
   // Adds a job to outstanding jobs list.
   void AddOutstandingJob(Job* job);
 
   // Returns the outstanding job for |key|, or NULL if there is none.
@@ skipping 47 matching lines @@
 
   // TODO(eroman): temp hack for http://crbug.com/15513
   bool shutdown_;
 
   DISALLOW_COPY_AND_ASSIGN(HostResolverImpl);
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_HOST_RESOLVER_IMPL_H_