RFR: 8344187: Remove SecurityManager and related calls from java.instrument [v2]

[Eirik Bjørsnøs]

On Mon, 18 Nov 2024 10:50:24 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Remove redundant SecurityManager, AccessController references
>> (following on from JDK-8338411: Implement JEP 486: Permanently Disable the Security Manager).
>> 
>> test/jdk/java/lang/instrument/ still passing.
>
> Kevin Walls has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
> 
>  - static block not needed
>  - Merge remote-tracking branch 'upstream/master' into 8344187_PostSM_instrument
>  - update
>  - 8344187: Remove SecurityManager and related calls from java.instrument

src/java.instrument/share/classes/sun/instrument/InstrumentationImpl.java line 40:

> 38: import java.net.URL;
> 39: import java.security.CodeSource;
> 40: import java.security.PrivilegedAction;

A note for future cleanups in this area: This `java.security.PrivilegedAction` import should probably have been removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22106#discussion_r1848231895