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

Unified Diff: content/renderer/render_view_impl.cc

Issue 30323002: [DRAFT] Create RenderFrameHostManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 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 side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 9a06f97085d87dc35076237263496a1df9216a27..16ad3f340e66669c601314a736578bad6d0cd202 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -960,6 +960,7 @@ void RenderViewImpl::Initialize(RenderViewImplParams* params) {
// The main frame WebFrame object is closed by
// RenderViewImpl::frameDetached().
webview()->setMainFrame(WebFrame::create(main_render_frame_.get()));
+ main_render_frame_->set_web_frame(webview()->mainFrame());
if (switches::IsTouchDragDropEnabled())
webview()->settings()->setTouchDragDropEnabled(true);
@@ -2157,7 +2158,11 @@ void RenderViewImpl::UpdateURL(WebFrame* frame) {
DCHECK(!navigation_state->history_list_was_cleared());
params.history_list_was_cleared = false;
- Send(new ViewHostMsg_FrameNavigate(routing_id_, params));
+ // Don't send this message while the subframe is swapped out.
+ // TODO(creis): This whole method should move to RenderFrame.
+ RenderFrameImpl* rf = RenderFrameImpl::FindByWebFrame(frame);
+ if (!rf->is_swapped_out())
+ Send(new ViewHostMsg_FrameNavigate(routing_id_, params));
}
last_page_id_sent_to_browser_ =
@@ -3927,6 +3932,12 @@ void RenderViewImpl::didFinishLoad(WebFrame* frame) {
FOR_EACH_OBSERVER(RenderViewObserver, observers_, DidFinishLoad(frame));
+ // Don't send this message while the subframe is swapped out.
+ // TODO(creis): This whole method should move to RenderFrame.
+ RenderFrameImpl* rf = RenderFrameImpl::FindByWebFrame(frame);
+ if (rf && rf->is_swapped_out())
+ return;
+
Send(new ViewHostMsg_DidFinishLoad(routing_id_,
frame->identifier(),
ds->request().url(),
@@ -5405,7 +5416,9 @@ void RenderViewImpl::NavigateToSwappedOutURL(blink::WebFrame* frame) {
// synchronously. Otherwise a new navigation can interrupt the navigation
// to kSwappedOutURL. If that happens to be to the page we had been
// showing, then WebKit will never send a commit and we'll be left spinning.
- CHECK(is_swapped_out_);
+ // TODO(creis): Until we move this to RenderFrame, we may call this from a
+ // swapped out RenderFrame while our own is_swapped_out_ is false.
+ //CHECK(is_swapped_out_);
GURL swappedOutURL(kSwappedOutURL);
WebURLRequest request(swappedOutURL);
frame->loadRequest(request);

Powered by Google App Engine
This is Rietveld 408576698