https://bugs.webkit.org/show_bug.cgi?id=64556
Source/WebCore:
Reviewed by Adam Barth.
Tests: http/tests/history/back-with-fragment-change.php
http/tests/navigation/navigation-interrupted-by-fragment.html
* loader/FrameLoader.cpp: Stop provisional load if a fragment commits.
* loader/HistoryController.cpp: Don't commit the wrong provisional item.
LayoutTests:
Reviewed by Adam Barth.
Required page-dismissal-modal-dialogs-iframe.html to be updated since
the dummy <a href="#"> link was clicked after the provisional load was
kicked off, thus causing it to to be aborted.
* fast/loader/page-dismissal-modal-dialogs.html:
* fast/loader/resources/page-dismissal-modal-dialogs-iframe.html:
* http/tests/history/back-with-fragment-change-expected.txt: Added.
* http/tests/history/back-with-fragment-change.php: Added.
* http/tests/history/resources/back-with-fragment-change-target.html: Added.
* http/tests/navigation/navigation-interrupted-by-fragment-expected.txt: Added.
* http/tests/navigation/navigation-interrupted-by-fragment.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@95259
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2011-09-15 Mihai Parparita <mihaip@chromium.org>
+
+ Fragment navigations should interrupt a provisional load of a different document
+ https://bugs.webkit.org/show_bug.cgi?id=64556
+
+ Reviewed by Adam Barth.
+
+ Required page-dismissal-modal-dialogs-iframe.html to be updated since
+ the dummy <a href="#"> link was clicked after the provisional load was
+ kicked off, thus causing it to to be aborted.
+
+ * fast/loader/page-dismissal-modal-dialogs.html:
+ * fast/loader/resources/page-dismissal-modal-dialogs-iframe.html:
+ * http/tests/history/back-with-fragment-change-expected.txt: Added.
+ * http/tests/history/back-with-fragment-change.php: Added.
+ * http/tests/history/resources/back-with-fragment-change-target.html: Added.
+ * http/tests/navigation/navigation-interrupted-by-fragment-expected.txt: Added.
+ * http/tests/navigation/navigation-interrupted-by-fragment.html: Added.
+
2011-09-15 Keishi Hattori <keishi@webkit.org>
[chromium] skipping tests due to r95188
}, false);
</script></head><body><div>
-<a href=# onclick="showMessages('mainFrame click')" id=anchor>anchor</a>
+<span onclick="showMessages('mainFrame click')" id=anchor>anchor</span>
<iframe src=resources/page-dismissal-modal-dialogs-iframe.html></iframe>
This page registers handlers for various types of unload events and attempts to
<html><head><meta charset=utf-8><title>Page Dismissal Modal Dialogs iFrame</title><script>
parent.registerEvents("iFrame", window, parent);
</script></head><body><div>
-<a href=# onclick="parent.showMessages('iFrame click')" id=anchor>anchor</a>
+<span onclick="parent.showMessages('iFrame click')" id=anchor>anchor</span>
</div></body></html>
--- /dev/null
+main frame - has 1 onunload handler(s)
+CONSOLE MESSAGE: line 19: Visited fragment and waited.
+PASS
+
+============== Back Forward List ==============
+ http://127.0.0.1:8000/history/back-with-fragment-change.php **nav target**
+ http://127.0.0.1:8000/history/resources/back-with-fragment-change-target.html **nav target**
+curr-> http://127.0.0.1:8000/history/resources/back-with-fragment-change-target.html#foo **nav target**
+===============================================
--- /dev/null
+<!--
+We intentionally want the page to load slowly (every time, hence no caching), so
+that when back-with-fragment-change-target.html calls history.back(), the load
+is provisional for a while (long enough for the window.location = '#foo' script
+to run and stop that load).
+-->
+<?php
+
+sleep(2);
+
+header("Cache-control: no-cache, no-store");
+header("Pragma: no-cache");
+?>
+
+<script>
+if (window.layoutTestController) {
+ layoutTestController.dumpBackForwardList();
+ layoutTestController.dumpAsText();
+ layoutTestController.waitUntilDone();
+}
+
+onload = function() {
+ if (sessionStorage.didNavigate) {
+ console.log('Should not have ended up back at the test start page');
+ delete sessionStorage.didNavigate;
+ if (window.layoutTestController)
+ layoutTestController.notifyDone();
+ return;
+ }
+
+ // Change the location in a timeout to make sure it generates a history entry
+ setTimeout(function() {
+ window.location = 'resources/back-with-fragment-change-target.html'
+ sessionStorage.didNavigate = true;
+ }, 0);
+}
+
+// Make sure there's no page cache.
+onunload = function() { };
+</script>
+<p>
+Tests that a history navigation that is aborted by a fragment change doesn't
+update the provisional history item. This test relies on
+<code>layoutTestController.dumpBackForwardList</code> to verify correctness
+and thus can only be run via DumpRenderTree.</p>
--- /dev/null
+<script>
+onload = function() {
+ setTimeout(function() {
+ // Start to go back (runs asynchonously)
+ history.back();
+ // But immediately cancel that load of a history item and navigate to
+ // a fragment on the page instead. We should remain on this page and
+ // a history item should be added (with past ones not being affected)
+ setTimeout(function(){window.location = '#foo'}, 0);
+ }, 0);
+}
+
+onhashchange = function() {
+ setTimeout(done, 100);
+}
+
+function done() {
+ delete sessionStorage.didNavigate;
+ console.log('Visited fragment and waited.');
+ if (window.layoutTestController) {
+ layoutTestController.notifyDone();
+ }
+}
+</script>
+PASS
--- /dev/null
+CONSOLE MESSAGE: line 23: First visit.
+CONSOLE MESSAGE: line 17: Visited fragment and waited.
+This test checks that interrupting a slow navigation with a fragment navigation will cancel the first navigation.
+
+============== Back Forward List ==============
+ http://127.0.0.1:8000/navigation/navigation-interrupted-by-fragment.html **nav target**
+curr-> http://127.0.0.1:8000/navigation/navigation-interrupted-by-fragment.html#abc **nav target**
+===============================================
--- /dev/null
+<script>
+if (window.layoutTestController) {
+ layoutTestController.dumpBackForwardList();
+ layoutTestController.dumpAsText();
+ layoutTestController.waitUntilDone();
+}
+onload = function() {
+ window.location = 'resources/slow-resource.pl?delay=100';
+ setTimeout(function() {window.location = 'navigation-interrupted-by-fragment.html#abc'}, 10);
+}
+
+onhashchange = function() {
+ setTimeout(done, 100);
+}
+
+function done() {
+ console.log('Visited fragment and waited.');
+ if (window.layoutTestController) {
+ layoutTestController.notifyDone();
+ }
+}
+
+console.log('First visit.');
+
+</script>
+<p>This test checks that interrupting a slow navigation with a fragment navigation will cancel the first navigation.
+2011-09-15 Mihai Parparita <mihaip@chromium.org>
+
+ Fragment navigations should interrupt a provisional load of a different document
+ https://bugs.webkit.org/show_bug.cgi?id=64556
+
+ Reviewed by Adam Barth.
+
+ Tests: http/tests/history/back-with-fragment-change.php
+ http/tests/navigation/navigation-interrupted-by-fragment.html
+
+ * loader/FrameLoader.cpp: Stop provisional load if a fragment commits.
+ * loader/HistoryController.cpp: Don't commit the wrong provisional item.
+
2011-09-15 Adrienne Walker <enne@google.com>
[chromium] Add temporary diagnostics for LayerTreeHost::commitTo crash
if (!shouldContinue)
return;
+ // If we have a provisional request for a different document, a fragment scroll should cancel it.
+ if (m_provisionalDocumentLoader && !equalIgnoringFragmentIdentifier(m_provisionalDocumentLoader->request().url(), request.url())) {
+ m_provisionalDocumentLoader->stopLoading();
+ setProvisionalDocumentLoader(0);
+ }
+
bool isRedirect = m_quickRedirectComing || policyChecker()->loadType() == FrameLoadTypeRedirectWithLockedBackForwardList;
loadInSameDocument(request.url(), 0, !isRedirect);
}
if (!m_provisionalItem)
return;
+ // The provisional item may represent a different pending navigation.
+ // Don't commit it if it isn't a same document navigation.
+ if (m_currentItem && !m_currentItem->shouldDoSameDocumentNavigationTo(m_provisionalItem.get()))
+ return;
+
// Commit the provisional item.
m_frameLoadComplete = false;
m_previousItem = m_currentItem;
projects / profile / ivi / webkit-efl.git / commitdiff