On Mon, 21 Sep 2026 19:14:35 GMT, Phil Race <[email protected]> wrote:
> PolyVertTest verifies the drawing of some de-generate polylines and polygons. > It has been problem-listed on all platforms for some time. > The test isn't particularly stable - problems with scaled UIs and rounded > window edges, and how it waits for the surface to be rendered. > I've updated it to fix all these things and save an image of any failed cases. > Also the test didn't specify any pipelines, so would run with the default. > On windows, where we use D3D if we can, but often fall back to software, this > can be misleading > as the test passes on software (since the same code is used for the reference > image) but fails with D3D. > > So we still need to problem-list the "default" case on windows, in case it is > D3D. > I also found that the default case fails with the metal pipeline on macOS. > Also although OpenGL passes on Windows & macOS it fails on Linux .. > > Cases that do pass are > - xrender=true (default) and xrender=false (x11) on Linux > - opengl=true on windows & macOS > - d3d=false on windows > > Although each platform has some failure mode, I don't think it can be called > a "generic" failure. > i.e I don't think it likely that the reason for an OpenGL failure on Linux is > the same as D3D on Windows. > So I've made sub-test cases, and will use separate bugs for each of the 3 > failure cases. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). test/jdk/ProblemList.txt line 216: > 214: sun/java2d/SunGraphics2D/DrawImageBilinear.java 8297175 linux-all > 215: sun/java2d/SunGraphics2D/PolyVertTest.java#default 8392800,8392802 > macosx-all,windows-all > 216: sun/java2d/SunGraphics2D/PolyVertTest.java#opengl 8392801 linux-all Interesting, this test passes on my HW running Ubuntu 24.04, but fails in VBox. test/jdk/sun/java2d/SunGraphics2D/PolyVertTest.java line 88: > 86: import java.awt.image.VolatileImage; > 87: import java.io.File; > 88: import java.io.IOException; Suggestion: import java.io.File; test/jdk/sun/java2d/SunGraphics2D/PolyVertTest.java line 294: > 292: return new Dimension(500, 500); > 293: } > 294: */ Suggestion: ? test/jdk/sun/java2d/SunGraphics2D/PolyVertTest.java line 648: > 646: if (frame != null) { > 647: //EventQueue.invokeAndWait(frame::dispose); > 648: } Suggestion: } It looks like this is already handled on line 371. test/jdk/sun/java2d/SunGraphics2D/PolyVertTest.java line 654: > 652: public static void createUI() { > 653: frame = new Frame("PolyVertTest"); > 654: frame = new Frame("SimplePrimQuality"); Suggestion: frame = new Frame("PolyVertTest"); ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32992#discussion_r4086474713 PR Review Comment: https://git.openjdk.org/jdk/pull/32992#discussion_r4086481243 PR Review Comment: https://git.openjdk.org/jdk/pull/32992#discussion_r4086483350 PR Review Comment: https://git.openjdk.org/jdk/pull/32992#discussion_r4086487340 PR Review Comment: https://git.openjdk.org/jdk/pull/32992#discussion_r4086464720
