First patch in making destructors of refcounted objects private.

base/message_loop_unittest.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 "base/logging.h"
 #include "base/message_loop.h"
 #include "base/platform_thread.h"
 #include "base/ref_counted.h"
 #include "base/thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 49 matching lines @@
   void Test2Mixed(const std::string& a, std::string* b) {
     ++test_count_;
     result_.append(a);
     result_.append(*b);
   }
 
   int test_count() const { return test_count_; }
   const std::string& result() const { return result_; }
 
  private:
+  friend class base::RefCounted<Foo>;
+
+  ~Foo() {}
+
   int test_count_;
   std::string result_;
 };
 
 class QuitMsgLoop : public base::RefCounted<QuitMsgLoop> {
  public:
   void QuitNow() {
     MessageLoop::current()->Quit();
   }
+
+ private:
+  friend class base::RefCounted<QuitMsgLoop>;
+
+  ~QuitMsgLoop() {}
 };
 
 void RunTest_PostTask(MessageLoop::Type message_loop_type) {
   MessageLoop loop(message_loop_type);
 
   // Add tests to message loop
   scoped_refptr<Foo> foo = new Foo();
   std::string a("a"), b("b"), c("c"), d("d");
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       foo.get(), &Foo::Test0));
@@ skipping 1398 matching lines @@
       message_loop.WatchFileDescriptor(fd,
           true, MessageLoopForIO::WATCH_WRITE, &controller, &delegate);
       controller.StopWatchingFileDescriptor();
     }
   }
   close(pipefds[0]);
   close(pipefds[1]);
 }
 
 #endif  // defined(OS_LINUX)