Bug 235407

Summary:

Load event never firing after form is submitted

Product:

WebKit

Reporter:

Sam Sneddon [:gsnedders] <gsnedders>

Component:

Page Loading

Assignee:

Chris Dumez <cdumez>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, ap, beidson, cdumez, darin, ews-watchlist, ggaren, japhet, webkit-bug-importer, youennf

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

URL:

http://wpt.live/html/semantics/forms/form-submission-target/rel-base-target.html

Attachments:

Description	Flags
WIP patch	none
Patch	none
Patch	ews-feeder: commit-queue-

Sam Sneddon [:gsnedders]

Reported 2022-01-20 08:48:17 PST

On the above test (note: this requires pop-ups to be allowed), the test is constantly resulting in a harness TIMEOUT. The underlying cause of this seems to be us never actually firing the load event for the test document.

Attachments
WIP patch (3.66 KB, patch) 2022-03-04 09:33 PST, Chris Dumez	no flags	Details Formatted Diff Diff
Patch (9.44 KB, patch) 2022-03-04 09:50 PST, Chris Dumez	no flags	Details Formatted Diff Diff
Patch (12.10 KB, patch) 2022-03-04 10:45 PST, Chris Dumez	ews-feeder: commit-queue-	Details Formatted Diff Diff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2022-01-20 08:48:29 PST

<rdar://problem/87831049>

Sam Sneddon [:gsnedders]

Comment 2 2022-01-21 04:00:31 PST

Reduces down to: <div id=log></div> <form action=about:blank target=_blank><input type=hidden name=input></form> <script> const t1 = performance.now(); const submitter = document.querySelector("form"); submitter.submit(); window.onload = () => { document.querySelector("#log").textContent = `loaded after ${performance.now() - t1}ms`; } </script>

Alexey Proskuryakov

Comment 3 2022-01-23 17:52:06 PST

I looked through our other form submission bugs, but this one seems new.

Chris Dumez

Comment 4 2022-03-04 09:10:52 PST

I am investigating.

Chris Dumez

Comment 5 2022-03-04 09:33:16 PST

Created attachment 453849 [details] WIP patch

Chris Dumez

Comment 6 2022-03-04 09:50:59 PST

Created attachment 453851 [details] Patch