Split the frame tree logic out of HistoryItem

Source/core/page/Page.cpp

Index: Source/core/page/Page.cpp
diff --git a/Source/core/page/Page.cpp b/Source/core/page/Page.cpp
index 052fb819415c90f52ae781ba686983b0623ebe27..7411d3f4430862bec1f20a20c0ac47d5b2cb949c 100644
--- a/Source/core/page/Page.cpp
+++ b/Source/core/page/Page.cpp
@@ -103,6 +103,7 @@ Page::Page(PageClients& pageClients)
     , m_contextMenuController(ContextMenuController::create(this, pageClients.contextMenuClient))
     , m_inspectorController(InspectorController::create(this, pageClients.inspectorClient))
     , m_pointerLockController(PointerLockController::create(this))
+    , m_history(adoptPtr(new HistoryController(this)))
     , m_settings(Settings::create(this))
     , m_progress(ProgressTracker::create())
     , m_backForwardClient(pageClients.backForwardClient)
@@ -280,13 +281,6 @@ void Page::setOpenedByDOM()
 
 void Page::goToItem(HistoryItem* item)
 {
-    // stopAllLoaders may end up running onload handlers, which could cause further history traversals that may lead to the passed in HistoryItem
-    // being deref()-ed. Make sure we can still use it with HistoryController::goToItem later.
-    RefPtr<HistoryItem> protector(item);
-
-    if (m_mainFrame->loader().history()->shouldStopLoadingForHistoryItem(item))
-        m_mainFrame->loader().stopAllLoaders();
-
     m_mainFrame->loader().history()->goToItem(item);
 }