Bug 272395 - [iOS] fast/writing-mode/english-bt-text-with-spelling-marker.html is a flaky failure after 277233@main
Summary: [iOS] fast/writing-mode/english-bt-text-with-spelling-marker.html is a flaky ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Wenson Hsieh
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2024-04-09 07:16 PDT by Wenson Hsieh
Modified: 2024-04-09 10:56 PDT (History)
4 users (show)

See Also:


Attachments
Image result before 277233@main (70.94 KB, image/png)
2024-04-09 07:27 PDT, Wenson Hsieh
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Wenson Hsieh 2024-04-09 07:16:50 PDT
.
Comment 1 Wenson Hsieh 2024-04-09 07:26:05 PDT
From the diff here:

https://build.webkit.org/results/Apple-iOS-17-Simulator-Debug-WK2-Tests/277233@main%20(1462)/fast/writing-mode/english-bt-text-with-spelling-marker-diffs.html

...it looks like after 277233@main, there's a tiny difference between the reference and test images.
Comment 2 Wenson Hsieh 2024-04-09 07:27:40 PDT
Created attachment 470839 [details]
Image result before 277233@main
Comment 3 Wenson Hsieh 2024-04-09 07:29:40 PDT
Attached an image showing the results (both reference and test) before my fix — there are no spelling dots at all.

My change in 277233@main basically made the test start working as intended on iOS 17 (i.e. showing spelling dots). I think we just need to adjust the fuzziness to accommodate.
Comment 4 Wenson Hsieh 2024-04-09 07:39:39 PDT
Pull request: https://github.com/WebKit/WebKit/pull/27029
Comment 5 EWS 2024-04-09 10:55:02 PDT
Committed 277254@main (ef7c999e5c10): <https://commits.webkit.org/277254@main>

Reviewed commits have been landed. Closing PR #27029 and removing active labels.
Comment 6 Radar WebKit Bug Importer 2024-04-09 10:56:15 PDT
<rdar://problem/126146299>