RefCounted types should not have public destructors, delegate cleanup

media/base/data_source.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.
 
 #ifndef MEDIA_BASE_DATA_SOURCE_H_
 #define MEDIA_BASE_DATA_SOURCE_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/time.h"
 #include "media/base/media_export.h"
 
 namespace media {
 
 class MEDIA_EXPORT DataSourceHost {
  public:
-  virtual ~DataSourceHost();
-
   // Set the total size of the media file.
   virtual void SetTotalBytes(int64 total_bytes) = 0;
 
   // Notify the host that byte range [start,end] has been buffered.
   // TODO(fischman): remove this method when demuxing is push-based instead of
   // pull-based.  http://crbug.com/131444
   virtual void AddBufferedByteRange(int64 start, int64 end) = 0;
 
   // Notify the host that time range [start,end] has been buffered.
   virtual void AddBufferedTimeRange(base::TimeDelta start,
                                     base::TimeDelta end) = 0;
+
+ protected:
+  virtual ~DataSourceHost();
 };
 
 class MEDIA_EXPORT DataSource : public base::RefCountedThreadSafe<DataSource> {
  public:
   typedef base::Callback<void(int64, int64)> StatusCallback;
   typedef base::Callback<void(int)> ReadCB;
   static const int kReadError;
 
   DataSource();
 
@@ skipping 33 matching lines @@
 
  private:
   DataSourceHost* host_;
 
   DISALLOW_COPY_AND_ASSIGN(DataSource);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_DATA_SOURCE_H_