On Fri, 8 Nov 2024 21:46:38 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
> Ths PR removes all remaining calls to `System::getSecurityManager` and the > `SecurityManager` class along with the `System.getGecurityManager() != null` > code paths. > > As part of doing this, the last uses of `PermissionHelper` and > `FXPermissions` were eliminated, so I deleted those two classes. Since > `PermissionHelper` was the last remaining use of `AccessControlException`, I > am adding JDK-8342998 to this PR. > > In a few places, I noted some "check permissions" methods that are now no-ops > that might be candidates for further cleanup (not directly related, so not > planned for jfx24). > > Finally, I deleted several unused methods in `MethodUtil` and `ReflectUtil` > (some of these were already removed in the corresponding JDK classes, and > some of them were unused after the changes to remove the call to > `getSecurityManager`). This pull request has now been integrated. Changeset: dd600658 Author: Kevin Rushforth <k...@openjdk.org> URL: https://git.openjdk.org/jfx/commit/dd60065808bc847d8efbb818b8f83c4e61290d29 Stats: 911 lines in 33 files changed: 10 ins; 866 del; 35 mod 8342997: Remove use of System::getSecurityManager and SecurityManager from JavaFX 8342998: Remove all uses of AccessControlException 8342994: Remove security manager calls in com.sun.javafx.reflect Reviewed-by: angorya, lkostyra ------------- PR: https://git.openjdk.org/jfx/pull/1631
