Second patch in making destructors of refcounted objects private.

net/proxy/single_threaded_proxy_resolver.cc

 // Copyright (c) 2009 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/proxy/single_threaded_proxy_resolver.h"
 
 #include "base/thread.h"
 #include "net/base/load_log.h"
 #include "net/base/net_errors.h"
 #include "net/proxy/proxy_info.h"
 
 namespace net {
 
 namespace {
 
 class PurgeMemoryTask : public base::RefCountedThreadSafe<PurgeMemoryTask> {
  public:
   explicit PurgeMemoryTask(ProxyResolver* resolver) : resolver_(resolver) {}
   void PurgeMemory() { resolver_->PurgeMemory(); }
  private:
+  friend class base::RefCountedThreadSafe<PurgeMemoryTask>;
+  ~PurgeMemoryTask() {}
   ProxyResolver* resolver_;
 };
 
 }
 
 // SingleThreadedProxyResolver::SetPacScriptTask ------------------------------
 
 // Runs on the worker thread to call ProxyResolver::SetPacScript.
 class SingleThreadedProxyResolver::SetPacScriptTask
     : public base::RefCountedThreadSafe<
@@ skipping 22 matching lines @@
     // Clear these to inform RequestComplete that it should not try to
     // access them.
     coordinator_ = NULL;
     callback_ = NULL;
   }
 
   // Returns true if Cancel() has been called.
   bool was_cancelled() const { return callback_ == NULL; }
 
  private:
+  friend class base::RefCountedThreadSafe<
+        SingleThreadedProxyResolver::SetPacScriptTask>;
+
+  ~SetPacScriptTask() {}
+
   // Runs on the worker thread.
   void DoRequest(ProxyResolver* resolver) {
     int rv = resolver->expects_pac_bytes() ?
         resolver->SetPacScriptByData(pac_bytes_, NULL) :
         resolver->SetPacScriptByUrl(pac_url_, NULL);
 
     DCHECK_NE(rv, ERR_IO_PENDING);
     origin_loop_->PostTask(FROM_HERE,
         NewRunnableMethod(this, &SetPacScriptTask::RequestComplete, rv));
   }
@@ skipping 57 matching lines @@
     // access them.
     coordinator_ = NULL;
     callback_ = NULL;
     results_ = NULL;
   }
 
   // Returns true if Cancel() has been called.
   bool was_cancelled() const { return callback_ == NULL; }
 
  private:
+  friend class base::RefCountedThreadSafe<SingleThreadedProxyResolver::Job>;
+
+  ~Job() {}
+
   // Runs on the worker thread.
   void DoQuery(ProxyResolver* resolver) {
     LoadLog* worker_log = new LoadLog;
     worker_log->AddRef();  // Balanced in QueryComplete.
 
     int rv = resolver->GetProxyForURL(url_, &results_buf_, NULL, NULL,
                                       worker_log);
     DCHECK_NE(rv, ERR_IO_PENDING);
 
     origin_loop_->PostTask(FROM_HERE,
@@ skipping 160 matching lines @@
   ProcessPendingJobs();
 }
 
 void SingleThreadedProxyResolver::RemoveOutstandingSetPacScriptTask(
     SetPacScriptTask* task) {
   DCHECK_EQ(outstanding_set_pac_script_task_.get(), task);
   outstanding_set_pac_script_task_ = NULL;
 }
 
 }  // namespace net