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

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 10808089: Merge 146897 - WebMediaPlayerImpl needs to own the audio source provider. (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1180/src/
Patch Set: Created 8 years, 5 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
===================================================================
--- chrome/renderer/chrome_content_renderer_client.cc (revision 147942)
+++ chrome/renderer/chrome_content_renderer_client.cc (working copy)
@@ -346,6 +346,7 @@
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) {
@@ -353,8 +354,8 @@
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(
« no previous file with comments | « chrome/renderer/chrome_content_renderer_client.h ('k') | chrome/renderer/prerender/prerender_webmediaplayer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698