Integrated: 8316206: Test StretchedFontTest.java fails for Baekmuk font
Alexey Ivanov
aivanov at openjdk.org
Fri Sep 22 12:50:29 UTC 2023
On Tue, 19 Sep 2023 13:41:27 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
> **Root cause**
>
> The _Baekmuk Headline_ font maps `\u6f22` (漢) to glyph id 16950 which has zero length.
>
> The test fails if the right half of the image with the rendered glyph contains only pixels of background colour. In this case, the left half of the image is also blank. It's somewhat false positive because of the broken font.
>
> **Fix**
>
> Ignore fonts which don't render the glyph correctly. If the left half of the image contains pixels of the background colour only, it is expected that the right half also contains background-coloured pixels only.
>
> The updated test does not fail with the _Baekmuk Headline_ font. It still detects the original problem and fails on builds without the fix for [JDK-8312555](https://bugs.openjdk.org/browse/JDK-8312555).
This pull request has now been integrated.
Changeset: 00f585bd
Author: Alexey Ivanov <aivanov at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/00f585bd22f527eca0107a9b4ed366f25754f0be
Stats: 27 lines in 1 file changed: 25 ins; 0 del; 2 mod
8316206: Test StretchedFontTest.java fails for Baekmuk font
Ignore broken fonts, i.e. the fonts for which
GlyphVector(TEXT).getVisualBounds().isEmpty() returns true
Reviewed-by: azvegint, prr, goetz
-------------
PR: https://git.openjdk.org/jdk/pull/15818
More information about the client-libs-dev
mailing list