Remove FrameTreeNode::ResetForNewProcess().
Currently, this method is simply calling ResetChildren() on the current
RenderFrameHost. It used to do more, but this is no longer the case.
This CL removes this method. Seeing RenderFrameHostImpl::ResetChildren()
in the code is more explicit about what happens. It is easier to
understand.
Bug: 609963
Change-Id: Ia82726f445b127ccca3758d7578e71eb32a49cbc
Reviewed-on: https://chromium-review.googlesource.com/c/1286421
Commit-Queue: Arthur Sonzogni <[email protected]>
Reviewed-by: Alex Moshchuk <[email protected]>
Cr-Commit-Position: refs/heads/master@{#600742}diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index b881bf4..759c8f4a 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -685,8 +685,8 @@
Source<WebContents>(this),
NotificationService::NoDetails());
- // Destroy all frame tree nodes except for the root; this notifies observers.
- frame_tree_.root()->ResetForNewProcess();
+ // Destroy all subframes now. This notifies observers.
+ GetMainFrame()->ResetChildren();
GetRenderManager()->ResetProxyHosts();
// Manually call the observer methods for the root frame tree node. It is
