RefCounted types should not have public destructors, base/ edition

base/threading/platform_thread.h

 // Copyright (c) 2012 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.
 
 // WARNING: You should *NOT* be using this class directly.  PlatformThread is
 // the low-level platform-specific abstraction to the OS's threading interface.
 // You should instead be using a message-loop driven Thread, see thread.h.
 
 #ifndef BASE_THREADING_PLATFORM_THREAD_H_
 #define BASE_THREADING_PLATFORM_THREAD_H_
@@ skipping 36 matching lines @@
   kThreadPriority_RealtimeAudio
 };
 
 // A namespace for low-level thread functions.
 class BASE_EXPORT PlatformThread {
  public:
   // Implement this interface to run code on a background thread.  Your
   // ThreadMain method will be called on the newly created thread.
   class BASE_EXPORT Delegate {
    public:
+    virtual void ThreadMain() = 0;
+
+   protected:
     virtual ~Delegate() {}
-    virtual void ThreadMain() = 0;
   };
 
   // Gets the current thread id, which may be useful for logging purposes.
   static PlatformThreadId CurrentId();
 
   // Yield the current thread so another thread can be scheduled.
   static void YieldCurrentThread();
 
   // Sleeps for the specified duration.
   static void Sleep(base::TimeDelta duration);
@@ skipping 41 matching lines @@
   static void SetThreadPriority(PlatformThreadHandle handle,
                                 ThreadPriority priority);
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(PlatformThread);
 };
 
 }  // namespace base
 
 #endif  // BASE_THREADING_PLATFORM_THREAD_H_