Changeset 173445 in webkit

Timestamp:

Sep 9, 2014 3:47:02 PM (10 years ago)

Author:

mitz@apple.com

Message:

FrameLoader::prepareForHistoryNavigation is unused
​https://bugs.webkit.org/show_bug.cgi?id=136683

Reviewed by Tim Horton.

• loader/FrameLoader.cpp:

(WebCore::FrameLoader::prepareForHistoryNavigation): Deleted.

• loader/FrameLoader.h: Removed the declaration.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/FrameLoader.cpp (modified) (1 diff)
• loader/FrameLoader.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-09-09  Dan Bernstein  <mitz@apple.com>
+
+        FrameLoader::prepareForHistoryNavigation is unused
+        https://bugs.webkit.org/show_bug.cgi?id=136683
+
+        Reviewed by Tim Horton.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::prepareForHistoryNavigation): Deleted.
+        * loader/FrameLoader.h: Removed the declaration.
+
 2014-09-09  Manuel Rego Casasnovas  <rego@igalia.com>
 

trunk/Source/WebCore/loader/FrameLoader.cpp

@@ -1161 +1161 @@
 }
 
-void FrameLoader::prepareForHistoryNavigation()
-{
-    // If there is no currentItem, but we still want to engage in 
-    // history navigation we need to manufacture one, and update
-    // the state machine of this frame to impersonate having
-    // loaded it.
-    RefPtr<HistoryItem> currentItem = history().currentItem();
-    if (!currentItem) {
-        currentItem = HistoryItem::create();
-        currentItem->setLastVisitWasFailure(true);
-        history().setCurrentItem(currentItem.get());
-        m_frame.page()->backForward().setCurrentItem(currentItem.get());
-
-        ASSERT(stateMachine().isDisplayingInitialEmptyDocument());
-        stateMachine().advanceTo(FrameLoaderStateMachine::DisplayingInitialEmptyDocumentPostCommit);
-        stateMachine().advanceTo(FrameLoaderStateMachine::CommittedFirstRealLoad);
-    }
-}
-
 void FrameLoader::prepareForLoadStart()
 {

trunk/Source/WebCore/loader/FrameLoader.h

@@ -103 +103 @@
     MixedContentChecker& mixedContentChecker() const { return m_mixedContentChecker; }
 
-    void prepareForHistoryNavigation();
     void setupForReplace();