Replace StreamParserHost interface with callbacks.

media/filters/chunk_demuxer.cc

Index: media/filters/chunk_demuxer.cc
diff --git a/media/filters/chunk_demuxer.cc b/media/filters/chunk_demuxer.cc
index 8b824494af8ab37781849bed501b8815a27d92d7..f414dc89788db64871bf9f19cdafef3d7e89c8c8 100644
--- a/media/filters/chunk_demuxer.cc
+++ b/media/filters/chunk_demuxer.cc
@@ -361,7 +361,9 @@ void ChunkDemuxer::Initialize(DemuxerHost* host,
 
     stream_parser_->Init(
         base::Bind(&ChunkDemuxer::OnStreamParserInitDone, this),
-        this);
+        base::Bind(&ChunkDemuxer::OnNewConfigs, this),

[scherkus (not reviewing), 2012/04/25 21:01:17]

just as a precautionary FYI for these types of changes...

before we weren't increasing the ref-count when passing in |this|, but
base::Bind() here is adding a ref

this can lead to circular refs / leaks if we're not careful

[acolwell GONE FROM CHROMIUM, 2012/04/25 22:17:02]

Good point. I've added base::Unretained so the code matches the original
ref-counting.

+        base::Bind(&ChunkDemuxer::OnAudioBuffers, this),
+        base::Bind(&ChunkDemuxer::OnVideoBuffers, this));
   }
 
   client_->DemuxerOpened(this);
@@ -708,7 +710,7 @@ bool ChunkDemuxer::OnNewConfigs(const AudioDecoderConfig& audio_config,
   return true;
 }
 
-bool ChunkDemuxer::OnAudioBuffers(const BufferQueue& buffers) {
+bool ChunkDemuxer::OnAudioBuffers(const StreamParser::BufferQueue& buffers) {
   if (!audio_.get())
     return false;
 
@@ -721,7 +723,7 @@ bool ChunkDemuxer::OnAudioBuffers(const BufferQueue& buffers) {
   return true;
 }
 
-bool ChunkDemuxer::OnVideoBuffers(const BufferQueue& buffers) {
+bool ChunkDemuxer::OnVideoBuffers(const StreamParser::BufferQueue& buffers) {
   if (!video_.get())
     return false;