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

Side by Side Diff: chrome/renderer/chrome_render_frame_observer.cc

Issue 117603002: Always create FrameTreeNodes and RenderFrameHosts for every frame. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix prerendering. Created 6 years, 11 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | content/browser/frame_host/frame_tree.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/renderer/chrome_render_frame_observer.h" 5 #include "chrome/renderer/chrome_render_frame_observer.h"
6 6
7 #include "chrome/common/prerender_messages.h" 7 #include "chrome/common/prerender_messages.h"
8 #include "chrome/renderer/prerender/prerender_helper.h" 8 #include "chrome/renderer/prerender/prerender_helper.h"
9 9
10 ChromeRenderFrameObserver::ChromeRenderFrameObserver( 10 ChromeRenderFrameObserver::ChromeRenderFrameObserver(
11 content::RenderFrame* render_frame) 11 content::RenderFrame* render_frame)
12 : content::RenderFrameObserver(render_frame) { 12 : content::RenderFrameObserver(render_frame) {
13 } 13 }
14 14
15 ChromeRenderFrameObserver::~ChromeRenderFrameObserver() { 15 ChromeRenderFrameObserver::~ChromeRenderFrameObserver() {
16 } 16 }
17 17
18 bool ChromeRenderFrameObserver::OnMessageReceived(const IPC::Message& message) { 18 bool ChromeRenderFrameObserver::OnMessageReceived(const IPC::Message& message) {
19 // Filter only. 19 // Filter only.
20 IPC_BEGIN_MESSAGE_MAP(ChromeRenderFrameObserver, message) 20 IPC_BEGIN_MESSAGE_MAP(ChromeRenderFrameObserver, message)
21 IPC_MESSAGE_HANDLER(PrerenderMsg_SetIsPrerendering, OnSetIsPrerendering); 21 IPC_MESSAGE_HANDLER(PrerenderMsg_SetIsPrerendering, OnSetIsPrerendering);
22 IPC_END_MESSAGE_MAP() 22 IPC_END_MESSAGE_MAP()
23 23
24 return false; 24 return false;
25 } 25 }
26 26
27 void ChromeRenderFrameObserver::OnSetIsPrerendering(bool is_prerendering) { 27 void ChromeRenderFrameObserver::OnSetIsPrerendering(bool is_prerendering) {
28 if (is_prerendering) { 28 if (is_prerendering) {
29 DCHECK(!prerender::PrerenderHelper::Get(render_frame())); 29 // If the PrerenderHelper for this frame already exists, don't create it. It
30 // can already be created for subframes during handling of
31 // RenderFrameCreated, if the parent frame was prerendering at time of
32 // subframe creation.
33 if (prerender::PrerenderHelper::Get(render_frame()))
34 return;
35
30 // The PrerenderHelper will destroy itself either after recording histograms 36 // The PrerenderHelper will destroy itself either after recording histograms
31 // or on destruction of the RenderView. 37 // or on destruction of the RenderView.
32 new prerender::PrerenderHelper(render_frame()); 38 new prerender::PrerenderHelper(render_frame());
33 } 39 }
34 } 40 }
OLDNEW
« no previous file with comments | « no previous file | content/browser/frame_host/frame_tree.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698