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

2024-11-14 Thread Lukasz Kostyra
On Wed, 13 Nov 2024 22:06:12 GMT, Kevin Rushforth 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`

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

2024-11-13 Thread Kevin Rushforth
> 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

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

2024-11-13 Thread Andy Goryachev
On Wed, 13 Nov 2024 15:53:49 GMT, Kevin Rushforth 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`

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

2024-11-13 Thread Kevin Rushforth
> 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

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

2024-11-13 Thread Andy Goryachev
On Wed, 13 Nov 2024 13:58:28 GMT, Kevin Rushforth 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`

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

2024-11-13 Thread Kevin Rushforth
> 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

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

2024-11-13 Thread Kevin Rushforth
> 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

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

2024-11-13 Thread Kevin Rushforth
On Tue, 12 Nov 2024 14:45:35 GMT, Kevin Rushforth wrote: >> modules/javafx.graphics/src/main/java/javafx/scene/input/Dragboard.java line >> 52: >> >>> 50: @Override >>> 51: Object getContentImpl(DataFormat dataFormat) { >>> 52: if (dataAccessRestricted) { >> >> this method can

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

2024-11-13 Thread Kevin Rushforth
On Wed, 13 Nov 2024 13:33:06 GMT, Kevin Rushforth 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`

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

2024-11-13 Thread Kevin Rushforth
On Fri, 8 Nov 2024 22:06:11 GMT, Andy Goryachev wrote: >> Kevin Rushforth has updated the pull request incrementally with three >> additional commits since the last revision: >> >> - Replace 'TODO: SM removal:' comments with 'TODO: JDK-nnn:' >> - Additional cleanup of unused fields, impor

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

2024-11-13 Thread Kevin Rushforth
> 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

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

2024-11-13 Thread Kevin Rushforth
On Mon, 11 Nov 2024 08:12:08 GMT, Lukasz Kostyra wrote: >> Kevin Rushforth has updated the pull request incrementally with three >> additional commits since the last revision: >> >> - Replace 'TODO: SM removal:' comments with 'TODO: JDK-nnn:' >> - Additional cleanup of unused fields, impo

Re: RFR: 8342997: Remove use of System::getSecurityManager and SecurityManager

2024-11-11 Thread Lukasz Kostyra
On Fri, 8 Nov 2024 21:46:38 GMT, Kevin Rushforth 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

Re: RFR: 8342997: Remove use of System::getSecurityManager and SecurityManager

2024-11-08 Thread Andy Goryachev
On Fri, 8 Nov 2024 21:46:38 GMT, Kevin Rushforth 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

RFR: 8342997: Remove use of System::getSecurityManager and SecurityManager

2024-11-08 Thread Kevin Rushforth
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. Sin