Skip to content

[ iOS Release ] http/tests/misc/embed-image-load-outlives-gc-without-crashing.html is a flakey timeout#60258

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
rniwa:fix295989-embed-image-load-outlives-gc-without-crashing-ios-flakiness
Mar 10, 2026
Merged

[ iOS Release ] http/tests/misc/embed-image-load-outlives-gc-without-crashing.html is a flakey timeout#60258
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
rniwa:fix295989-embed-image-load-outlives-gc-without-crashing-ios-flakiness

Conversation

@rniwa
Copy link
Member

@rniwa rniwa commented Mar 10, 2026

bd9598e

[ iOS Release ] http/tests/misc/embed-image-load-outlives-gc-without-crashing.html is a flakey timeout
https://bugs.webkit.org/show_bug.cgi?id=295989
rdar://155885979

Reviewed by Chris Dumez.

Based on Claude's analysis, this test flakiness comes from requestAnimationFrame sometimes not firing in iOS.
On iOS, requestAnimationFrame callbacks only fire when the compositor schedules a rendering update, which may
or may not happen after a call to requestAnimationFrame in the test.

Try replacing requestAnimationFrame with setTimeout to make it more reliable.

* LayoutTests/http/tests/misc/embed-image-load-outlives-gc-without-crashing.html:

Canonical link: https://commits.webkit.org/308977@main

327ea4e

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows Apple Internal
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe 🛠 win ✅ 🛠 ios-apple
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 🧪 win-tests loading 🛠 mac-apple
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe ⏳ 🛠 vision-apple
✅ 🧪 ios-wk2-wpt ✅ 🧪 api-mac-debug ✅ 🛠 gtk3-libwebrtc
✅ 🧪 api-ios ✅ 🧪 mac-wk1 ✅ 🛠 gtk
✅ 🧪 mac-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 api-gtk
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🛠 playstation
✅ 🛠 🧪 unsafe-merge ✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 tv
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@rniwa rniwa self-assigned this Mar 10, 2026
@rniwa rniwa added the Images For bugs in image handling. label Mar 10, 2026
Copy link
Contributor

@cdumez cdumez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it's worth trying though I'm surprised we're not unskipping the test here.

@rniwa rniwa added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 10, 2026
…crashing.html is a flakey timeout

https://bugs.webkit.org/show_bug.cgi?id=295989
rdar://155885979

Reviewed by Chris Dumez.

Based on Claude's analysis, this test flakiness comes from requestAnimationFrame sometimes not firing in iOS.
On iOS, requestAnimationFrame callbacks only fire when the compositor schedules a rendering update, which may
or may not happen after a call to requestAnimationFrame in the test.

Try replacing requestAnimationFrame with setTimeout to make it more reliable.

* LayoutTests/http/tests/misc/embed-image-load-outlives-gc-without-crashing.html:

Canonical link: https://commits.webkit.org/308977@main
@webkit-commit-queue webkit-commit-queue force-pushed the fix295989-embed-image-load-outlives-gc-without-crashing-ios-flakiness branch from 327ea4e to bd9598e Compare March 10, 2026 15:56
@webkit-commit-queue
Copy link
Collaborator

Committed 308977@main (bd9598e): https://commits.webkit.org/308977@main

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

@webkit-commit-queue webkit-commit-queue merged commit bd9598e into WebKit:main Mar 10, 2026
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 10, 2026
@rniwa rniwa deleted the fix295989-embed-image-load-outlives-gc-without-crashing-ios-flakiness branch March 10, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Images For bugs in image handling.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants