WebMediaPlayerImpl needs to own the audio source provider.

webkit/media/webmediaplayer_impl.cc

Index: webkit/media/webmediaplayer_impl.cc
diff --git a/webkit/media/webmediaplayer_impl.cc b/webkit/media/webmediaplayer_impl.cc
index 5707a7b9ac76dfb1a93dbb8c71621e93665a1145..fb6738f712d059e8c6e625faa0414b34f063c4ed 100644
--- a/webkit/media/webmediaplayer_impl.cc
+++ b/webkit/media/webmediaplayer_impl.cc
@@ -107,6 +107,7 @@ WebMediaPlayerImpl::WebMediaPlayerImpl(
     base::WeakPtr<WebMediaPlayerDelegate> delegate,
     media::FilterCollection* collection,
     WebKit::WebAudioSourceProvider* audio_source_provider,
+    media::AudioRendererSink* audio_renderer_sink,
     media::MessageLoopFactory* message_loop_factory,
     MediaStreamClient* media_stream_client,
     media::MediaLog* media_log)
@@ -130,6 +131,7 @@ WebMediaPlayerImpl::WebMediaPlayerImpl(
       accelerated_compositing_reported_(false),
       incremented_externally_allocated_memory_(false),
       audio_source_provider_(audio_source_provider),
+      audio_renderer_sink_(audio_renderer_sink),
       is_local_source_(false) {
   media_log_->AddEvent(

[scherkus (not reviewing), 2012/07/09 23:13:23]

can you DCHECK_EQ() that the two pointers are equal + comment why?

[Raymond Toy, 2012/07/10 02:51:39]

No, I can't do that because they're pointers derived from two different classes.
 I even tried casting each to void*, but they're different, due to how classes
are laid out.

[scherkus (not reviewing), 2012/07/10 17:13:51]

Right -- that makes sense :\

       media_log_->CreateEvent(media::MediaLogEvent::WEBMEDIAPLAYER_CREATED));