Second patch in making destructors of refcounted objects private.

net/http/http_cache_unittest.cc

 // Copyright (c) 2006-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/http/http_cache.h"
 
 #include "base/hash_tables.h"
 #include "base/message_loop.h"
 #include "base/scoped_vector.h"
 #include "base/string_util.h"
@@ skipping 43 matching lines @@
       DCHECK(slash != std::string::npos);
       url = GURL(key.substr(slash + 1));
     } else {
       url = GURL(key);
     }
     const MockTransaction* t = FindMockTransaction(url);
     DCHECK(t);
     test_mode_ = t->test_mode;
   }
 
-  ~MockDiskEntry() {
-  }
-
   bool is_doomed() const { return doomed_; }
 
   virtual void Doom() {
     doomed_ = true;
   }
 
   virtual void Close() {
     Release();
   }
 
@@ skipping 165 matching lines @@
     // The pending operation is already in the message loop (and hopefuly
     // already in the second pass).  Just notify the caller that it finished.
     CallbackLater(completion_callback, 0);
     return net::ERR_IO_PENDING;
   }
 
   // Fail most subsequent requests.
   void set_fail_requests() { fail_requests_ = true; }
 
  private:
+  friend class base::RefCounted<MockDiskEntry>;
+
+  ~MockDiskEntry() {}
+
   // Unlike the callbacks for MockHttpTransaction, we want this one to run even
   // if the consumer called Close on the MockDiskEntry.  We achieve that by
   // leveraging the fact that this class is reference counted.
   void CallbackLater(net::CompletionCallback* callback, int result) {
     MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(this,
         &MockDiskEntry::RunCallback, callback, result));
   }
   void RunCallback(net::CompletionCallback* callback, int result) {
     if (busy_) {
       // This is kind of hacky, but controlling the behavior of just this entry
@@ skipping 3148 matching lines @@
   std::string headers;
   response.headers->GetNormalizedHeaders(&headers);
 
   EXPECT_EQ("HTTP/1.1 200 OK\n"
             "Date: Wed, 22 Jul 2009 03:15:26 GMT\n"
             "Last-Modified: Wed, 06 Feb 2008 22:38:21 GMT\n",
             headers);
 
   RemoveMockTransaction(&mock_network_response);
 }