Bug 223005 – [BigSur arm64] 4 canvas-color-fonts tests consistently failing

• Status: RESOLVED
• Resolution: FIXED
• Component: Canvas

[Ryan Haddad]

The following tests has been consistently failing on the Big Sur arm64 bots since they were brought up:

fast/text/canvas-color-fonts/stroke-color-COLR.html
fast/text/canvas-color-fonts/stroke-color-shadow-COLR.html
http/tests/canvas/color-fonts/stroke-color-sbix.html
http/tests/canvas/color-fonts/stroke-color-shadow-sbix.html

https://build.webkit.org/results/Apple-BigSur-Release-AppleSilicon-WK2-Tests/r274168%20(116)/results.html

https://results.webkit.org/?suite=layout-tests&suite=layout-tests&suite=layout-tests&suite=layout-tests&suite=layout-tests&suite=layout-tests&suite=layout-tests&suite=layout-tests&test=fast%2Ftext%2Fcanvas-color-fonts%2Fstroke-color-COLR.html&test=fast%2Ftext%2Fcanvas-color-fonts%2Fstroke-color-shadow-COLR.html&test=http%2Ftests%2Fcanvas%2Fcolor-fonts%2Fstroke-color-sbix.html&test=http%2Ftests%2Fcanvas%2Fcolor-fonts%2Fstroke-color-shadow-sbix.html&test=fast%2Ftext%2Fcanvas-color-fonts%2Fstroke-color-COLR.html&test=fast%2Ftext%2Fcanvas-color-fonts%2Fstroke-color-shadow-COLR.html&test=http%2Ftests%2Fcanvas%2Fcolor-fonts%2Fstroke-color-sbix.html&test=http%2Ftests%2Fcanvas%2Fcolor-fonts%2Fstroke-color-shadow-sbix.html

[Radar WebKit Bug Importer]

<rdar://problem/75239330>

[Ryan Haddad]

Marked tests as failing in https://trac.webkit.org/changeset/274188/webkit

[Cameron McCormack (:heycam)]

Created attachment 432357 [details]
Patch

[Cameron McCormack (:heycam)]

Created attachment 432362 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 432362 [details]
Patch

How did they pass before?

[Cameron McCormack (:heycam)]

(In reply to Simon Fraser (smfr) from comment #5)
> How did they pass before?

Because there are bugs stroking text with anti-aliasing turned off when accelerated canvas drawing is enabled on macOS.  There are two bugs I've noticed -- one is that on ARM anti-aliasing is unexpectedly used, which is the reason these tests were marked as failing on ARM originally.  The other is that on Intel, requests to stroke at 1px end up being stroked at 2px.  I'll file some bugs here to track.

(These tests don't run on platforms that don't support color fonts.)

[Cameron McCormack (:heycam)]

(In reply to Cameron McCormack (:heycam) from comment #6)
> Because there are bugs stroking text with anti-aliasing turned off when
> accelerated canvas drawing is enabled on macOS.  There are two bugs I've
> noticed -- one is that on ARM anti-aliasing is unexpectedly used, which is
> the reason these tests were marked as failing on ARM originally.

Filed bug 227465 for that.

> The other is that on Intel, requests to stroke at 1px end up being stroked
> at 2px.

This I'm not sure whether it's a bug.  I think we end up with a 2px thick stroke because the 1px stroke straddles pixel boundaries, and the non-anti-aliased stroke covers both pixels.  Maybe that's the correct thing to do.  Though it is a difference from the non-accelerated canvas rendering behavior.

[Cameron McCormack (:heycam)]

(In reply to Cameron McCormack (:heycam) from comment #7)
> This I'm not sure whether it's a bug.  I think we end up with a 2px thick
> stroke because the 1px stroke straddles pixel boundaries, and the
> non-anti-aliased stroke covers both pixels.

And by "think" I mean I tried changing the test to render the text at a 0.5px offset and the result was a 1px thick stroke.

[EWS]

Committed r279356 (239224@main): <https://commits.webkit.org/239224@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 432362 [details].