On Wed, 13 Nov 2024 13:58:28 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`). > > Kevin Rushforth has updated the pull request incrementally with one > additional commit since the last revision: > > remove redundant override of Dragboard::getContentImpl modules/javafx.graphics/src/main/java/com/sun/javafx/tk/quantum/WindowStage.java line 596: > 594: } > 595: > 596: // TODO: JDK-8344111: Consider removing this obsolete method what method? there is no method, only the field L597 ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1631#discussion_r1840648086
