Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10572)

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 10662030: WebMediaPlayerImpl needs to own the audio source provider. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 76f6020969c2504889fe504f5bafd0fa5bf05e4f..98dd70f0f0aa51ac269a55a96312280a8a4f5654 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -342,6 +342,7 @@ ChromeContentRendererClient::OverrideCreateWebMediaPlayer(
base::WeakPtr<webkit_media::WebMediaPlayerDelegate> delegate,
media::FilterCollection* collection,
WebKit::WebAudioSourceProvider* audio_source_provider,
+ media::AudioRendererSink* audio_renderer_sink,
media::MessageLoopFactory* message_loop_factory,
webkit_media::MediaStreamClient* media_stream_client,
media::MediaLog* media_log) {
@@ -349,8 +350,8 @@ ChromeContentRendererClient::OverrideCreateWebMediaPlayer(
return NULL;
return new prerender::PrerenderWebMediaPlayer(render_view, frame, client,
- delegate, collection, audio_source_provider, message_loop_factory,
- media_stream_client, media_log);
+ delegate, collection, audio_source_provider, audio_renderer_sink,
+ message_loop_factory, media_stream_client, media_log);
}
WebPlugin* ChromeContentRendererClient::CreatePlugin(

Powered by Google App Engine
This is Rietveld 408576698