Bug 248014 - [WinCairo][WK1] Can't paint a font after painting very large font
Summary: [WinCairo][WK1] Can't paint a font after painting very large font
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-16 19:32 PST by Fujii Hironori
Modified: 2022-12-22 23:32 PST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 2022-11-16 19:32:30 PST 
[WinCairo][WK1] Can't paint a font after painting very large font

For example, fast/text/letter-spacing-produces-nan-width.html contains a very large font.
After load the test case, fast/text/simple-lines-float.html fails.

> python3.exe ./Tools/Scripts/run-webkit-tests --wincairo --debug -1 --no-retry-failures fast/text/letter-spacing-produces-nan-width.html fast/text/simple-lines-float.html --iterations=2 -v

> [1/4] fast/text/simple-lines-float.html passed
> [2/4] fast/text/letter-spacing-produces-nan-width.html passed
> [3/4] fast/text/simple-lines-float.html failed unexpectedly (reference mismatch maxDifference=255; totalPixels=1356)
> [4/4] fast/text/letter-spacing-produces-nan-width.html passed

The first test run of simple-lines-float.html passed, but failed in the second because it was after letter-spacing-produces-nan-width.html.

See also:
  Bug 246844 – REGRESSION(cairo 1.17.6)[WinCairo] rendering glitches of twitter.com and yahoo.co.jp
Comment 2 Fujii Hironori 2022-11-17 12:06:38 PST 
WinCairo WK1 is reporting the following flaky test failures. They seems to be caused by this bug.

Unexpected flakiness: image-only failures (5)
  fast/css3-text/css3-text-decoration/text-decoration-style-double-space-scales.html [ ImageOnlyFailure Pass ]
  fast/text/mobileasset-font.html [ ImageOnlyFailure Pass ]
  fast/text/multi-code-unit-simple-path.html [ ImageOnlyFailure Pass ]
  fast/text/simple-lines-float.html [ ImageOnlyFailure Pass ]
  fast/text/text-underline-position-subscript.html [ ImageOnlyFailure Pass ]
Comment 3 Fujii Hironori 2022-11-20 13:02:48 PST 
cairo issue ticket:
[windows] Don't leave cairo_font_face_t in an error state even after very large cairo_scaled_font_t creation (#607) · Issues · cairo / cairo · GitLab
https://gitlab.freedesktop.org/cairo/cairo/-/issues/607