Title: [173445] trunk/Source/WebCore
- Revision
- 173445
- Author
- [email protected]
- Date
- 2014-09-09 15:47:02 -0700 (Tue, 09 Sep 2014)
Log 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.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (173444 => 173445)
--- trunk/Source/WebCore/ChangeLog 2014-09-09 21:58:11 UTC (rev 173444)
+++ trunk/Source/WebCore/ChangeLog 2014-09-09 22:47:02 UTC (rev 173445)
@@ -1,3 +1,14 @@
+2014-09-09 Dan Bernstein <[email protected]>
+
+ 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 <[email protected]>
[CSS Grid Layout] Ignore ::first-letter pseudo-element
Modified: trunk/Source/WebCore/loader/FrameLoader.cpp (173444 => 173445)
--- trunk/Source/WebCore/loader/FrameLoader.cpp 2014-09-09 21:58:11 UTC (rev 173444)
+++ trunk/Source/WebCore/loader/FrameLoader.cpp 2014-09-09 22:47:02 UTC (rev 173445)
@@ -1160,25 +1160,6 @@
frame->loader().m_isComplete = false;
}
-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()
{
m_progressTracker->progressStarted();
Modified: trunk/Source/WebCore/loader/FrameLoader.h (173444 => 173445)
--- trunk/Source/WebCore/loader/FrameLoader.h 2014-09-09 21:58:11 UTC (rev 173444)
+++ trunk/Source/WebCore/loader/FrameLoader.h 2014-09-09 22:47:02 UTC (rev 173445)
@@ -102,7 +102,6 @@
IconController& icon() const { return *m_icon; }
MixedContentChecker& mixedContentChecker() const { return m_mixedContentChecker; }
- void prepareForHistoryNavigation();
void setupForReplace();
// FIXME: These are all functions which start loads. We have too many.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
