On Wed, 7 Oct 2026 15:18:09 GMT, Andy Goryachev <[email protected]> wrote:
>> Marius Hanl has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 13 commits: >> >> - change references in test and comments >> - Move isEmbedded(), and move the SWT check as well into GlassPlatform >> - Merge branch 'master' of https://github.com/openjdk/jfx into >> 8365309-move-PlatformUtil-to-javafx.graphics >> - Merge branch 'master' of https://github.com/openjdk/jfx into >> 8365309-move-PlatformUtil-to-javafx.graphics >> - proper getter now >> - Move platform factory to GlassPlatform >> - Move accessible handling to GlassPlatform >> - Move headless, egl embedded type (now monocle) to GlassPlatform >> - rename to GlassPlatform, improve code a bit >> - revert back >> - ... and 3 more: https://git.openjdk.org/jfx/compare/68b0b112...3d429bb6 > > modules/javafx.graphics/src/main/java/com/sun/javafx/tk/quantum/QuantumToolkit.java > line 163: > >> 161: >> 162: static final boolean liveResize = ((Supplier<Boolean>) () -> { >> 163: String result = (PlatformUtil.isMac() || >> PlatformUtil.isWindows()) && !GlassPlatform.isSWT() ? "true" : "false"; > > I think we should get rid of all these `Supplier`s since we removed the > `AccessController`. yes, same thought. Shall I do in this PR? Otherwise I will create a ticket and clean it up afterwards. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2299#discussion_r4208637740
