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

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 8095007: Defer loading of audio/video tags while prerendering. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Addressing Jam's comments. Created 9 years 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
« no previous file with comments | « chrome/renderer/chrome_content_renderer_client.h ('k') | chrome/renderer/prerender/prerender_helper.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 495414a732da96ba15105fe49cf59078812b4e66..202d5e09a07341b47d094593a666a064e3746402 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -49,6 +49,7 @@
#include "chrome/renderer/plugins/missing_plugin.h"
#include "chrome/renderer/plugins/plugin_uma.h"
#include "chrome/renderer/prerender/prerender_helper.h"
+#include "chrome/renderer/prerender/prerender_webmediaplayer.h"
#include "chrome/renderer/print_web_view_helper.h"
#include "chrome/renderer/renderer_histogram_snapshots.h"
#include "chrome/renderer/safe_browsing/malware_dom_details.h"
@@ -272,6 +273,25 @@ bool ChromeContentRendererClient::OverrideCreatePlugin(
return true;
}
+bool ChromeContentRendererClient::OverrideCreateWebMediaPlayer(
+ content::RenderView* render_view,
+ WebKit::WebMediaPlayerClient* client,
+ base::WeakPtr<webkit_media::WebMediaPlayerDelegate> delegate,
+ media::FilterCollection* collection,
+ media::MessageLoopFactory* message_loop_factory,
+ webkit_media::MediaStreamClient* media_stream_client,
+ media::MediaLog* media_log,
+ webkit_media::WebMediaPlayerImpl** player) {
+ if (!prerender::PrerenderHelper::IsPrerendering(render_view)) {
+ *player = NULL;
jam 2011/12/05 22:40:17 nit: since this is an override function and you're
Shishir 2011/12/05 22:46:20 Done.
+ return false;
+ }
+ *player = new prerender::PrerenderWebMediaPlayer(render_view, client,
+ delegate, collection, message_loop_factory, media_stream_client,
+ media_log);
+ return true;
+}
+
WebPlugin* ChromeContentRendererClient::CreatePlugin(
content::RenderView* render_view,
WebFrame* frame,
« no previous file with comments | « chrome/renderer/chrome_content_renderer_client.h ('k') | chrome/renderer/prerender/prerender_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698