Bug 264751 - REGRESSION(268639@main): [ macOS wk2 ] fast/frames/iframe-window-focus.html is a flaky failure
Summary: REGRESSION(268639@main): [ macOS wk2 ] fast/frames/iframe-window-focus.html i...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Frames (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Charlie Wolfe
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-11-13 10:01 PST by Ben Schwartz
Modified: 2023-11-14 16:02 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Schwartz 2023-11-13 10:01:20 PST
fast/frames/iframe-window-focus.html

This test is consistently failing on most macOS wk2 configurations.

HISTORY:

https://results.webkit.org/?suite=layout-tests&test=fast%2Fframes%2Fiframe-window-focus.html&flavor=wk2&platform=mac

TEXT DIFF:

 This tests that focusing an editable iframe's window works correctly and allows text and newlines to be entered.
 
-TEST
+EST
 PASS

DIFF URL:

https://build.webkit.org/results/Apple-Sonoma-Release-WK2-Tests/270636%40main%20(623)/fast/frames/iframe-window-focus-pretty-diff.html

REPRODUCIBILITY:

I was able to reproduce this issue on macOS Sonoma (14.1, 23B74/arm64) ToT using the following command:

run-webkit-tests --clobber-old-results --verbose --iterations 100 --exit-after-n-failures 1 fast/frames/iframe-window-focus.html

REGRESSION:

I am unable to bisect an exact regression point, as there were a string of build failures spanning the range in which we would expect it to be (268626@main - 268640@main). My best guess is that 268632@main caused the failure, as changes seem like they could be related.
Comment 1 Radar WebKit Bug Importer 2023-11-13 10:01:43 PST
<rdar://problem/118340493>
Comment 2 EWS 2023-11-13 10:10:25 PST
Test gardening commit 270644@main (34368464f184): <https://commits.webkit.org/270644@main>

Reviewed commits have been landed. Closing PR #20424 and removing active labels.
Comment 3 Gerald Squelart 2023-11-13 15:49:38 PST
Thank you Ben for the initial report and bisection.
I was able to fix the compilation issue locally, and further bisect the regression.
The regression point is actually https://commits.webkit.org/268639@main aka https://bugs.webkit.org/show_bug.cgi?id=262286 rdar://116167166

The failing test sends keyDown events, and the 268639@main patch is about key events, that seems to confirm my bisection.
I'll ping the author to investigate further...
Comment 4 Ben Schwartz 2023-11-14 09:45:57 PST
Thanks Gerald! I modified the title to reflect your findings.
Comment 5 Charlie Wolfe 2023-11-14 11:16:52 PST
Pull request: https://github.com/WebKit/WebKit/pull/20496
Comment 6 EWS 2023-11-14 16:02:51 PST
Committed 270731@main (7326eec80566): <https://commits.webkit.org/270731@main>

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