[jdk17] RFR: 8268349: Provide clear run-time warnings about Security Manager deprecation [v4]

[Sean Mullan]

On Thu, 17 Jun 2021 14:27:30 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> test/jdk/java/security/ProtectionDomain/RecursionDebug.java line 91:
>> 
>>> 89:         }
>>> 90: 
>>> 91:         System.setSecurityManager(null);
>> 
>> Why did this line need to be removed?
>
> This is where the `setSecurityManager` method is called again when a security manager has already been installed, and the `getProtectionDomain` permission is needed. While I can add the new permission into the policy file, my understanding is that this line was just a clean-up and the test was about implementation of `ProtectionDomain::toString` (esp the `seeAllp` method called by it) when debug is on.

Ok, thanks for the explanation. I think it should be ok to remove this line then.

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

PR: https://git.openjdk.java.net/jdk17/pull/13