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

Unified Diff: content/browser/frame_host/frame_tree.cc

Issue 666563005: Set up Mojo connection when RenderFrameHost is reused for new RenderFrame (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 2 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: content/browser/frame_host/frame_tree.cc
diff --git a/content/browser/frame_host/frame_tree.cc b/content/browser/frame_host/frame_tree.cc
index 27ed57ff1a32d0cfdabf81af901f6e28b2e0d496..99a6ca9d392e5b398fad66c1e2dcbf9cd0ce3c62 100644
--- a/content/browser/frame_host/frame_tree.cc
+++ b/content/browser/frame_host/frame_tree.cc
@@ -230,6 +230,10 @@ void FrameTree::RenderProcessGone(RenderViewHost* render_view_host) {
// ForEach does not add a node's children to the queue until after visiting
// the node itself.
ForEach(base::Bind(&ResetNodesForNewProcess, render_view_host));
+
+ // Ensure that if the main frame is reused for a new RenderFrame, it will
+ // set up the Mojo connection with that frame.
+ GetMainFrame()->InvalidateMojoConnection();
Charlie Reis 2014/10/23 18:38:31 I don't think we want to do this on the main frame
blundell 2014/10/24 14:42:54 Done.
}
RenderFrameHostImpl* FrameTree::GetMainFrame() const {

Powered by Google App Engine
This is Rietveld 408576698