RefCounted types should not have public destructors, Linux fixes

chrome/browser/jankometer.cc

 // Copyright (c) 2011 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 <limits>
 
 #include "chrome/browser/jankometer.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 203 matching lines @@
 //------------------------------------------------------------------------------
 class IOJankObserver : public base::RefCountedThreadSafe<IOJankObserver>,
                        public MessageLoopForIO::IOObserver,
                        public MessageLoop::TaskObserver {
  public:
   IOJankObserver(const char* thread_name,
                  TimeDelta excessive_duration,
                  bool watchdog_enable)
       : helper_(thread_name, excessive_duration, watchdog_enable) {}
 
-  ~IOJankObserver() {}
-
   // Attaches the observer to the current thread's message loop. You can only
   // attach to the current thread, so this function can be invoked on another
   // thread to attach it.
   void AttachToCurrentThread() {
     MessageLoop::current()->AddTaskObserver(this);
     MessageLoopForIO::current()->AddIOObserver(this);
   }
 
   // Detaches the observer to the current thread's message loop.
   void DetachFromCurrentThread() {
@@ skipping 19 matching lines @@
     helper_.StartProcessingTimers(queueing_time);
   }
 
   virtual void DidProcessTask(base::TimeTicks time_posted) OVERRIDE {
     helper_.EndProcessingTimers();
   }
 
  private:
   friend class base::RefCountedThreadSafe<IOJankObserver>;
 
+  ~IOJankObserver() {}
+
   JankObserverHelper helper_;
 
   DISALLOW_COPY_AND_ASSIGN(IOJankObserver);
 };
 
 //------------------------------------------------------------------------------
 class UIJankObserver : public base::RefCountedThreadSafe<UIJankObserver>,
                        public MessageLoop::TaskObserver,
                        public MessageLoopForUI::Observer {
  public:
@@ skipping 144 matching lines @@
     delete ui_observer;
     ui_observer = NULL;
   }
   if (io_observer) {
     // IO thread can't be running when we remove observers.
     DCHECK((!g_browser_process) || !(g_browser_process->io_thread()));
     delete io_observer;
     io_observer = NULL;
   }
 }