RefCounted types should not have public destructors, delegate cleanup

media/base/demuxer.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_DEMUXER_H_
 #define MEDIA_BASE_DEMUXER_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/time.h"
 #include "media/base/data_source.h"
 #include "media/base/demuxer_stream.h"
 #include "media/base/media_export.h"
 #include "media/base/pipeline_status.h"
 
 namespace media {
 
 class MEDIA_EXPORT DemuxerHost : public DataSourceHost {
  public:
-  virtual ~DemuxerHost();
-
   // Get the duration of the media in microseconds.  If the duration has not
   // been determined yet, then returns 0.
   virtual void SetDuration(base::TimeDelta duration) = 0;
 
   // Sets the byte offset at which the client is requesting the video.
   virtual void SetCurrentReadPosition(int64 offset) = 0;
 
   // Stops execution of the pipeline due to a fatal error.  Do not call this
   // method with PIPELINE_OK.
   virtual void OnDemuxerError(PipelineStatus error) = 0;
+
+ protected:
+  virtual ~DemuxerHost();
 };
 
 class MEDIA_EXPORT Demuxer : public base::RefCountedThreadSafe<Demuxer> {
  public:
   Demuxer();
 
   // Completes initialization of the demuxer.
   //
   // The demuxer does not own |host| as it is guaranteed to outlive the
   // lifetime of the demuxer. Don't delete it!
@@ skipping 33 matching lines @@
   friend class base::RefCountedThreadSafe<Demuxer>;
   virtual ~Demuxer();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(Demuxer);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_DEMUXER_H_