On Mon, 28 Oct 2024 15:18:45 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> Removes doPrivileged in the following module: >> >> - javafx.swt >> >> See JDK-8342441 for details. >> >> As a helpful hint for reviewers, I recommend reviewing this using the "Hide >> whitespace" option. > > modules/javafx.swt/src/main/java/javafx/embed/swt/FXCanvas.java line 359: > >> 357: }); >> 358: return null; >> 359: }, null, FXCANVAS_PERMISSION); > > I think `FXCANVAS_PERMISSION` is unused after this and can be removed (along > with the related import). there is a duplicate FXCANVAS_PERMISSION in PlatformImpl which should also be removed. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1611#discussion_r1823154534
