Bug 307351 – [MacOS] imported/w3c/web-platform-tests/html/browsers/browsing-the-web/navigating-across-documents/replace-before-load/form-requestsubmit.html is a flaky text failure.

• Component: New Bugs
• Status: NEW

[Diego De La Toba]

imported/w3c/web-platform-tests/html/browsers/browsing-the-web/navigating-across-documents/replace-before-load/form-requestsubmit.html is a flaky text failure on MacOS across all architectures. 


HISTORY:
https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fhtml%2Fbrowsers%2Fbrowsing-the-web%2Fnavigating-across-documents%2Freplace-before-load%2Fform-requestsubmit.html


DIFF:
--- /Volumes/Data/worker/Apple-Tahoe-Release-AppleSilicon-WK2-Tests/build/layout-test-results/imported/w3c/web-platform-tests/html/browsers/browsing-the-web/navigating-across-documents/replace-before-load/form-requestsubmit-expected.txt
+++ /Volumes/Data/worker/Apple-Tahoe-Release-AppleSilicon-WK2-Tests/build/layout-test-results/imported/w3c/web-platform-tests/html/browsers/browsing-the-web/navigating-across-documents/replace-before-load/form-requestsubmit-actual.txt
@@ -1,4 +1,4 @@
 
 
-PASS Replace before load, triggered by formElement.requestSubmit()
+FAIL Replace before load, triggered by formElement.requestSubmit() assert_equals: expected "http://web-platform.test:8800/common/blank.html?thereplacement=" but got "http://web-platform.test:8800/html/browsers/browsing-the-web/navigating-across-documents/replace-before-load/resources/code-injector.html?pipe=sub(none)&code=%0A%20%20%20%20const%20form%20%3D%20document.createElement(%22form%22)%3B%0A%20%20%20%20form.action%20%3D%20%22%2Fcommon%2Fblank.html%22%3B%0A%0A%20%20%20%20const%20input%20%3D%20document.createElement(%22input%22)%3B%0A%20%20%20%20input.type%20%3D%20%22hidden%22%3B%0A%20%20%20%20input.name%20%3D%20%22thereplacement%22%3B%0A%20%20%20%20form.append(input)%3B%0A%0A%20%20%20%20document.currentScript.before(form)%3B%0A%20%20%20%20form.requestSubmit()%3B%0A%20%20"
 

DIFF URL:
https://build.webkit.org/results/Apple-Tahoe-Release-AppleSilicon-WK2-Tests/307071@main%20(1538)/imported/w3c/web-platform-tests/html/browsers/browsing-the-web/navigating-across-documents/replace-before-load/form-requestsubmit-pretty-diff.html


REPRODUCTION:
I was able to reproduce this on macOS Tahoe ToT with the following:

run-webkit-tests --force --release --no-retry --iterations 2000 imported/w3c/web-platform-tests/html/browsers/browsing-the-web/navigating-across-documents/replace-before-load/form-requestsubmit.html

I am going to mark expectations as pass fail while this pends investigation.

[Radar WebKit Bug Importer]

<rdar://problem/169977453>

[Diego De La Toba]

Pull request: https://github.com/WebKit/WebKit/pull/58217

[EWS]

Test gardening commit 307088@main (46162187e0db): <https://commits.webkit.org/307088@main>

Reviewed commits have been landed. Closing PR #58217 and removing active labels.

[Diego De La Toba]

The test uses waitForLoad() with { once: true } (helpers.js:9), which only listens for the first load event. Form submission SHOULD replace navigation before code-injector.html finishes loading however, there could be a possible race condition where code-injector.html finishes loading first, fires a load event, and the form submission doesn't replace it.

[Diego De La Toba]

Pull request: https://github.com/WebKit/WebKit/pull/58223