First patch in making destructors of refcounted objects private.

base/message_pump_glib_unittest.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 "base/message_pump_glib.h"
 
 #include <gtk/gtk.h>
 #include <math.h>
 
 #include <algorithm>
@@ skipping 312 matching lines @@
     } else {
       injector_->AddEvent(
           0, NewRunnableMethod(this, &ConcurrentHelper::FromEvent));
     }
   }
 
   int event_count() const { return event_count_; }
   int task_count() const { return task_count_; }
 
  private:
+  friend class base::RefCounted<ConcurrentHelper>;
+
+  ~ConcurrentHelper() {}
+
   static const int kStartingEventCount = 20;
   static const int kStartingTaskCount = 20;
 
   EventInjector* injector_;
   int event_count_;
   int task_count_;
 };
 
 }  // namespace
 
@@ skipping 105 matching lines @@
 
   void Quit() {
     quit_ = true;
   }
 
   void Reset() {
     quit_ = false;
   }
 
  private:
+  friend class base::RefCounted<GLibLoopRunner>;
+
+  ~GLibLoopRunner() {}
+
   bool quit_;
 };
 
 void TestGLibLoopInternal(EventInjector* injector) {
   // Allow tasks to be processed from 'native' event loops.
   MessageLoop::current()->SetNestableTasksAllowed(true);
   scoped_refptr<GLibLoopRunner> runner = new GLibLoopRunner();
 
   int task_count = 0;
   // Add a couple of dummy events
@@ skipping 66 matching lines @@
 
 TEST_F(MessagePumpGLibTest, TestGtkLoop) {
   // Tests that events and posted tasks are correctly exectuted if the message
   // loop is not run by MessageLoop::Run() but by a straight Gtk loop.
   // Note that in this case we don't make strong guarantees about niceness
   // between events and posted tasks.
   loop()->PostTask(FROM_HERE,
                    NewRunnableFunction(TestGtkLoopInternal, injector()));
   loop()->Run();
 }