Skip to content

Commit

Permalink
Handle the case where a new navigation happens synchronously inside R…
Browse files Browse the repository at this point in the history
…enderFrameImpl::CommitSameDocumentNavigation

This is broken after https://chromium-review.googlesource.com/c/chromium/src/+/4300672

Bug: 1427449
Change-Id: Ie6ec0a597eeb45e3a7ae11215fc010d284a997bf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4375397
Commit-Queue: Nate Chapin <japhet@chromium.org>
Reviewed-by: Rakina Zata Amni <rakina@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1123066}
  • Loading branch information
natechapin authored and chromium-wpt-export-bot committed Mar 28, 2023
1 parent 390d915 commit 94aec98
Showing 1 changed file with 15 additions and 0 deletions.
@@ -0,0 +1,15 @@
<!doctype html>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<script src="../navigation-methods/return-value/resources/helpers.js"></script>
<script>
promise_test(async t => {
// Wait for after the load event so that the navigation doesn't get converted
// into a replace navigation.
await new Promise(resolve => window.onload = () => t.step_timeout(resolve, 0));

await navigation.navigate("#push").finished;
navigation.onnavigate = () => history.replaceState(null, "", "#");
await assertBothRejectDOM(t, navigation.back(), "AbortError");
}, "replaceState inside a navigate event for navigation.back()");
</script>

0 comments on commit 94aec98

Please sign in to comment.