Skip to content

REGRESSION:(311911@main) [macOS]fonts/font-cache-memory-pressure-crash.html is a flaky text failure.#63755

Merged
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
cdumez:313282_test_rebaseline
Apr 28, 2026
Merged

REGRESSION:(311911@main) [macOS]fonts/font-cache-memory-pressure-crash.html is a flaky text failure.#63755
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
cdumez:313282_test_rebaseline

Conversation

@cdumez
Copy link
Copy Markdown
Contributor

@cdumez cdumez commented Apr 28, 2026

404e188

REGRESSION:(311911@main) [macOS]fonts/font-cache-memory-pressure-crash.html is a flaky text failure.
https://bugs.webkit.org/show_bug.cgi?id=313282
rdar://175554270

Reviewed by Ryosuke Niwa.

311911@main updated the user agent style sheet to mark `option` and
`optgroup` as `display: block`. This caused this fuzzer-generated test
to become flaky with 2 extra blank lines, presumably due to option or
optgroup elements in the DOM tree that now are displayed as block
instead of inline.

To address the flakiness, restore the old behavior for this test by
marking `option` and `optgroup` as `display: inline` in the <style>
at the beginning of the test.

* LayoutTests/fonts/font-cache-memory-pressure-crash.html:
* LayoutTests/platform/mac-wk2/TestExpectations:

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

040546f

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 ⏳ 🛠 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
✅ 🛠 🧪 merge ✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🛠 playstation
✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 tv
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@cdumez cdumez self-assigned this Apr 28, 2026
@cdumez cdumez added the New Bugs Unclassified bugs are placed in this component until the correct component can be determined. label Apr 28, 2026
Test passes if it doesn't crash.


undefined
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Where is this "undefined" coming from? I see one option element, but unsure what's "undefined" there. And I don't see any such output when opening the test directly in Chrome.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't check (the test is a huge fuzzer generated one), "undefined" was already there. The is the existing LayoutTests/fonts/font-cache-memory-pressure-crash-expected.txt:

Test passes if it doesn't crash.
undefined

The only thing my patch did in 311911@main was add 2 extra blank lines by changing a css stylesheet.

That said, it looks like it is failing on EWS so maybe the 2 new blank lines are only on a very specific config :( I'll look.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah right, rebaselining a flaky test wouldn't work :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Right, the radar had said consistent failure but looking at the bots, it was in fact flaky.

@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Apr 28, 2026
@cdumez cdumez removed the merging-blocked Applied to prevent a change from being merged label Apr 28, 2026
@cdumez cdumez changed the title REGRESSION:(311911@main) [macOS]fonts/font-cache-memory-pressure-crash.html is a constant text failure. REGRESSION:(311911@main) [macOS]fonts/font-cache-memory-pressure-crash.html is a flaky text failure. Apr 28, 2026
@cdumez cdumez force-pushed the 313282_test_rebaseline branch from 3598bd4 to a6432ae Compare April 28, 2026 04:28
@cdumez cdumez requested review from annevk and rniwa April 28, 2026 06:58
@cdumez cdumez force-pushed the 313282_test_rebaseline branch from a6432ae to 040546f Compare April 28, 2026 07:51
@cdumez cdumez added safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks merge-queue Applied to send a pull request to merge-queue and removed safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks labels Apr 28, 2026
…h.html is a flaky text failure.

https://bugs.webkit.org/show_bug.cgi?id=313282
rdar://175554270

Reviewed by Ryosuke Niwa.

311911@main updated the user agent style sheet to mark `option` and
`optgroup` as `display: block`. This caused this fuzzer-generated test
to become flaky with 2 extra blank lines, presumably due to option or
optgroup elements in the DOM tree that now are displayed as block
instead of inline.

To address the flakiness, restore the old behavior for this test by
marking `option` and `optgroup` as `display: inline` in the <style>
at the beginning of the test.

* LayoutTests/fonts/font-cache-memory-pressure-crash.html:
* LayoutTests/platform/mac-wk2/TestExpectations:

Canonical link: https://commits.webkit.org/312192@main
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 312192@main (404e188): https://commits.webkit.org/312192@main

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

@webkit-commit-queue webkit-commit-queue merged commit 404e188 into WebKit:main Apr 28, 2026
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New Bugs Unclassified bugs are placed in this component until the correct component can be determined.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants