On Tue, 19 Sep 2023 13:41:27 GMT, Alexey Ivanov <aiva...@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 <aiva...@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
