On Mon, 7 Jul 2025 09:29:22 GMT, Johan Vos <j...@openjdk.org> wrote: >> After spending a year in the sandbox repository, the Headless Platform is >> now ready to be reviewed in the main repository. >> >> ### the Headless Platform >> The Headless Platform is a top-level com.sun.glass.ui platform that replaces >> the second-level Monocle-Headless subplatform, that is part of the top-level >> Monocle platform. >> The platform can be used like any other platform, especially for running >> headless JavaFX applications, or for running tests (e.g. on CI systems) >> >> ### changes >> The code for the Headless Platform is in a new package >> com.sun.glass.ui.headless in the javafx.graphics module, and it does not >> require a code change in other packages. >> This PR adds a simple change in the `build.gradle` file, to make the >> Headless Platform the standard when running headless tests (instead of using >> Monocle/Headless) >> >> ### enable the Headless Platform >> Setting the system property `glass.platform` to `Headless` will select the >> Headless Platform instead of the default one (either gtk, mac or win). >> >> ### testing >> `gradlew --info -PHEADLESS_TEST=true -PFULL_TEST=true :systemTests:cleanTest >> :systemTests:test` >> runs all the system tests, apart from the robot tests. There are 2 failing >> tests, but there are valid reasons for those to fail. >> >> ### robot tests >> Most of the robot tests are working on headless as well. add `-PUSE_ROBOT` >> to test those. > > Johan Vos has updated the pull request incrementally with one additional > commit since the last revision: > > Process more reviewer comments
modules/javafx.graphics/src/main/java/com/sun/glass/ui/headless/HeadlessWindow.java line 296: > 294: int b = rgba & 0xFF; > 295: > 296: Color color = Color.rgb(r, g, b, a); `Color::rgb` requires opacity between 0 and 1, therefore, you need: Color color = Color.rgb(r, g, b, a / 255.0); Minor: you can inline the variable. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1836#discussion_r2190667156
