First patch in making destructors of refcounted objects private.

media/filters/ffmpeg_demuxer.h

 // 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.
 
 // Implements the Demuxer interface using FFmpeg's libavformat.  At this time
 // will support demuxing any audio/video format thrown at it.  The streams
 // output mime types audio/x-ffmpeg and video/x-ffmpeg and include an integer
 // key FFmpegCodecID which contains the CodecID enumeration value.  The CodecIDs
 // can be used to create and initialize the corresponding FFmpeg decoder.
 //
@@ skipping 36 matching lines @@
 
 // Forward declaration for scoped_ptr_malloc.
 class ScopedPtrAVFree;
 
 class FFmpegDemuxerStream : public DemuxerStream, public AVStreamProvider {
  public:
   // Maintains a reference to |demuxer| and initializes itself using information
   // inside |stream|.
   FFmpegDemuxerStream(FFmpegDemuxer* demuxer, AVStream* stream);
 
-  virtual ~FFmpegDemuxerStream();
-
   // Returns true is this stream has pending reads, false otherwise.
   //
   // Safe to call on any thread.
   virtual bool HasPendingReads();
 
   // Enqueues and takes ownership over the given AVPacket, returns the timestamp
   // of the enqueued packet.
   virtual base::TimeDelta EnqueuePacket(AVPacket* packet);
 
   // Signals to empty the buffer queue and mark next packet as discontinuous.
   virtual void FlushBuffers();
 
   // Empties the queues and ignores any additional calls to Read().
   virtual void Stop();
 
   // Returns the duration of this stream.
   virtual base::TimeDelta duration() { return duration_; }
 
   // DemuxerStream implementation.
   virtual const MediaFormat& media_format();
   virtual void Read(Callback1<Buffer*>::Type* read_callback);
 
   // AVStreamProvider implementation.
   virtual AVStream* GetAVStream() { return stream_; }
 
  protected:
   virtual void* QueryInterface(const char* interface_id);
 
  private:
+  virtual ~FFmpegDemuxerStream();
+
   // Carries out enqueuing a pending read on the demuxer thread.
   void ReadTask(Callback1<Buffer*>::Type* read_callback);
 
   // Attempts to fulfill a single pending read by dequeueing a buffer and read
   // callback pair and executing the callback.
   void FulfillPendingRead();
 
   // Converts an FFmpeg stream timestamp into a base::TimeDelta.
   base::TimeDelta ConvertTimestamp(int64 timestamp);
 
@@ skipping 128 matching lines @@
   // If true, then it's our first seek and we won't call av_read_frame().  It's
   // a hack to get around some issue with FFmpeg.
   bool first_seek_hack_;
 
   DISALLOW_COPY_AND_ASSIGN(FFmpegDemuxer);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_FILTERS_FFMPEG_DEMUXER_H_