RESOLVED FIXED 229975
fast/text/FontFaceSet-status-after-style-update.html is flaky. 
https://bugs.webkit.org/show_bug.cgi?id=229975
Summary fast/text/FontFaceSet-status-after-style-update.html is flaky.
Carlos Alberto Lopez Perez
Reported 2021-09-06 16:47:30 PDT
The test fast/text/FontFaceSet-status-after-style-update.html is flaky since it was added on r282015 See: https://results.webkit.org/?suite=layout-tests&test=fast%2Ftext%2FFontFaceSet-status-after-style-update.html When it fails it outputs this diff: --- /Volumes/Data/worker/bigsur-release-applesilicon-tests-wk2/build/layout-test-results/fast/text/FontFaceSet-status-after-style-update-expected.txt +++ /Volumes/Data/worker/bigsur-release-applesilicon-tests-wk2/build/layout-test-results/fast/text/FontFaceSet-status-after-style-update-actual.txt @@ -4,9 +4,10 @@ PASS fonts.status is "loaded" -PASS fonts.status is "loading" +FAIL fonts.status should be loading. Was loaded. PASS fonts.status is "loaded" PASS successfullyParsed is true +Some tests failed. TEST COMPLETE
Attachments
Patch (3.31 KB, patch)
2021-09-07 23:49 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (3.29 KB, patch)
2021-09-07 23:50 PDT, Myles C. Maxfield 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Carlos Alberto Lopez Perez
Comment 1 2021-09-06 18:57:01 PDT
Updated expectations on r282071
Myles C. Maxfield
Comment 2 2021-09-07 14:34:45 PDT
Thanks for the test expectations fix. I'll work on this bug this week.
Myles C. Maxfield
Comment 3 2021-09-07 16:20:04 PDT
I'm first seeing if I can reproduce this with run-webkit-tests --child-processes=1 --test-list
Myles C. Maxfield
Comment 4 2021-09-07 23:06:33 PDT
This sequence of tests seems to reproduce the bug 100%: fast/shapes/shape-outside-floats/shape-outside-clip-path-selection.html fast/text/FontFaceSet-check-after-style-update.html fast/text/FontFaceSet-status-after-style-update.html
Myles C. Maxfield
Comment 5 2021-09-07 23:15:43 PDT
This reproduces the problem: DYLD_FRAMEWORK_PATH=/Users/mmaxfield/Build/Products/Debug ~/Build/Products/Debug/WebKitTestRunner --no-timeout file:///Users/mmaxfield/src/WebKit/OpenSource/LayoutTests/fast/shapes/shape-outside-floats/shape-outside-clip-path-selection.html file:///Users/mmaxfield/src/WebKit/OpenSource/LayoutTests/fast/text/FontFaceSet-check-after-style-update.html file:///Users/mmaxfield/src/WebKit/OpenSource/LayoutTests/fast/text/FontFaceSet-status-after-style-update.html
Myles C. Maxfield
Comment 6 2021-09-07 23:36:28 PDT
Even though we're calling internals.clearMemoryCache(), the CSSFontFaceSource is still coming to us in the Success state.
Myles C. Maxfield
Comment 7 2021-09-07 23:49:35 PDT
Created attachment 437602 [details] Patch
Myles C. Maxfield
Comment 8 2021-09-07 23:50:37 PDT
Created attachment 437603 [details] Patch
Myles C. Maxfield
Comment 9 2021-09-07 23:51:02 PDT
I think this counts as test gardening, so I'll just land this if EWS is all green.
EWS
Comment 10 2021-09-08 12:44:13 PDT
Tools/Scripts/svn-apply failed to apply attachment 437603 [details] to trunk. Please resolve the conflicts and upload a new patch.
Myles C. Maxfield
Comment 11 2021-09-08 12:47:41 PDT
Committed r282163 (241455@main): <https://commits.webkit.org/241455@main>
Radar WebKit Bug Importer
Comment 12 2021-09-08 12:48:17 PDT
<rdar://problem/82886587>
Myles C. Maxfield
Comment 13 2021-09-08 17:40:08 PDT
*** Bug 230062 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.