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

Side by Side Diff: chrome/renderer/prerender/prerender_webmediaplayer.cc

Issue 8095007: Defer loading of audio/video tags while prerendering. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Fixing comment. Created 9 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/renderer/prerender/prerender_webmediaplayer.h"
6
7 #include "chrome/common/render_messages.h"
8 #include "content/public/renderer/render_view.h"
9 #include "media/base/filter_collection.h"
10 #include "media/base/media_log.h"
11 #include "media/base/pipeline_impl.h"
12 #include "third_party/WebKit/Source/WebKit/chromium/public/WebURL.h"
13 #include "webkit/glue/webmediaplayer_delegate.h"
14
15 namespace prerender {
16
17 PrerenderWebMediaPlayer::PrerenderWebMediaPlayer(
18 content::RenderView* render_view,
19 WebKit::WebMediaPlayerClient* client,
20 base::WeakPtr<webkit_glue::WebMediaPlayerDelegate> delegate,
21 media::FilterCollection* collection,
22 media::MessageLoopFactory* message_loop_factory,
23 webkit_glue::MediaStreamClient* media_stream_client,
24 media::MediaLog* media_log)
25 : RenderViewObserver(render_view),
26 WebMediaPlayerImpl(client,
27 delegate,
28 collection,
29 message_loop_factory,
30 media_stream_client,
31 media_log),
32 is_prerendering_(true) {
33 }
34
35 PrerenderWebMediaPlayer::~PrerenderWebMediaPlayer() {}
36
37 void PrerenderWebMediaPlayer::load(const WebKit::WebURL& url) {
38 if (is_prerendering_) {
39 url_to_load_.reset(new WebKit::WebURL(url));
40 return;
41 }
42 WebMediaPlayerImpl::load(url);
43 }
44
45 void PrerenderWebMediaPlayer::cancelLoad() {
46 if (is_prerendering_) {
47 url_to_load_.reset(NULL);
48 return;
49 }
50 WebMediaPlayerImpl::cancelLoad();
51 }
52
53 bool PrerenderWebMediaPlayer::OnMessageReceived(const IPC::Message& message) {
54 IPC_BEGIN_MESSAGE_MAP(PrerenderWebMediaPlayer, message)
55 IPC_MESSAGE_HANDLER(ChromeViewMsg_SetIsPrerendering, OnSetIsPrerendering)
56 IPC_END_MESSAGE_MAP()
57
58 return false;
59 }
60
61 void PrerenderWebMediaPlayer::OnSetIsPrerendering(bool is_prerendering) {
62 // Prerendering can only be enabled prior to a RenderView's first
63 // navigation, so no PrerenderWebMediaPlayer should see the notification
64 // that enables prerendering.
65 DCHECK(!is_prerendering);
66 if (is_prerendering_ && !is_prerendering) {
67 is_prerendering_ = false;
68 if (url_to_load_.get()) {
dominich 2011/11/14 19:40:35 nit: remove these braces
Shishir 2011/11/14 19:51:53 Done.
69 load(*url_to_load_);
70 }
71 }
72 }
73
74 } // namespace prerender
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698