RFR: 8342997: Remove use of System::getSecurityManager and SecurityManager [v4]

Andy Goryachev angorya at openjdk.org
Wed Nov 13 15:49:49 UTC 2024


On Wed, 13 Nov 2024 13:58:28 GMT, Kevin Rushforth <kcr at 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


More information about the openjfx-dev mailing list