RFR: JDK-8344303 : Remove usage of URLUtil.getConnectPermission from sun.awt.SunToolkit and sun.awt.image.URLImageSource [v2]

Harshitha Onkar honkar at openjdk.org
Fri Nov 22 19:53:16 UTC 2024


On Fri, 22 Nov 2024 19:32:26 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   removed java.desktop
>
> src/java.desktop/share/classes/sun/awt/image/URLImageSource.java line 48:
> 
>> 46:             try {
>> 47:                 java.security.Permission perm =
>> 48:                       URLUtil.getConnectPermission(u);
> 
> `URLUtil.getConnectPermission(u);` is no longer used anywhere and can be removed
> 
> 
> https://github.com/openjdk/jdk/commit/8c644ca310c44995a1fc899eabb5611b79873c68#diff-50e0953b57d7ba2dbb8faa9b1766b99234d5ae4cb271c48ece13b9e3d0c9ca89
> 
> so the jdk/src/java.base/share/classes/sun/net/util/URLUtil.java part can be reverted

Will changes **sun/net/util/URLUtil.java** be handled in [JDK-8344180](https://bugs.openjdk.org/browse/JDK-8344180)? Or do we do it here?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22330#discussion_r1854562822


More information about the client-libs-dev mailing list