On Tue, 26 Nov 2024 20:10:10 GMT, Phil Race <[email protected]> wrote:
> Remove SecurityManager related code from sun.awt.image classes Changes LGTM Did a 1st pass through the changes and build looks good. src/java.desktop/share/classes/sun/awt/image/URLImageSource.java line 85: > 83: // host/port that the URL is redirected to. More than > that > 84: // and we give up and just throw a SecurityException. > 85: // This is not directly related to SecurityManager so > keep it. Looks like a valid reason to retain SecurityException in this case. ------------- Marked as reviewed by honkar (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22398#pullrequestreview-2462756128 PR Review Comment: https://git.openjdk.org/jdk/pull/22398#discussion_r1859281576
