NEW 231316
Regression (r281291): [ iOS 15 ] fast/text/accessibility-bold-system-font-2.html is image failing
https://bugs.webkit.org/show_bug.cgi?id=231316
Summary Regression (r281291): [ iOS 15 ] fast/text/accessibility-bold-system-font-2.h...
ayumi_kojima
Reported 2021-10-06 11:44:46 PDT
fast/text/accessibility-bold-system-font-2.html Is constantly image failing on iOS 15 Debug and flaky (very flaky) image failing on iOS 15 Release. History: https://results.webkit.org/?suite=layout-tests&test=fast%2Ftext%2Faccessibility-bold-system-font-2.html According to the history, it started failing after updating to iOS 15.
Attachments
image diff (12.68 KB, image/png)
2021-10-06 11:45 PDT, ayumi_kojima
no flags
Patch (10.24 KB, patch)
2022-04-13 00:28 PDT, Myles C. Maxfield
no flags
ayumi_kojima
Comment 2 2021-10-06 11:45:36 PDT
Created attachment 440398 [details] image diff
Radar WebKit Bug Importer
Comment 3 2021-10-06 11:45:58 PDT
ayumi_kojima
Comment 4 2021-10-06 11:50:10 PDT
ayumi_kojima
Comment 5 2021-10-06 11:55:05 PDT
I was able to reproduce the image failure locally on iOS 15 sim using run-webkit-tests --force --debug --ios-simulator fast/text/accessibility-bold-system-font-2.html
ayumi_kojima
Comment 6 2021-10-06 13:56:43 PDT
As I was bisecting, the test failed on r281291 and newer but passed on r281290 and older.
Myles C. Maxfield
Comment 7 2022-04-13 00:28:21 PDT
Darin Adler
Comment 8 2022-04-18 17:29:52 PDT
Comment on attachment 457510 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=457510&action=review > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:392 > - auto& after = keyframes[i]; > + auto& after = keyframes[i + 1]; Great fix, but not mentioned in the change log. Do we have test coverage? > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:417 > - auto& after = keyframes[i]; > + auto& after = keyframes[i + 1]; Ditto.
Myles C. Maxfield
Comment 9 2022-04-18 22:29:07 PDT
Comment on attachment 457510 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=457510&action=review >> Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:392 >> + auto& after = keyframes[i + 1]; > > Great fix, but not mentioned in the change log. Do we have test coverage? This patch is not marked for review 🤔 We do; I’m handling it in https://bugs.webkit.org/show_bug.cgi?id=239374. That bug has a big chain of blocking bugs because I’m trying to figure out a performance regression, and I’m lumping in the fix in with that effort.
Brent Fulgham
Comment 10 2022-06-23 14:29:50 PDT
This actually seems to be tracked by: rdar://88249241.
Note You need to log in before you can comment on or make changes to this bug.