RFR: 8277457: AccessControlException: access denied ("java.net.NetPermission" "getCookieHandler") [v2]

[Kevin Rushforth]

On Wed, 24 Nov 2021 23:40:01 GMT, Jose Pereda <jpereda at openjdk.org> wrote:

>> Kevin Rushforth has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address review feedback: use method reference instead of lambda
>
> modules/javafx.web/src/main/java/com/sun/webkit/network/CookieJar.java line 47:
> 
>> 45:         @SuppressWarnings("removal")
>> 46:         CookieHandler handler =
>> 47:             AccessController.doPrivileged((PrivilegedAction<CookieHandler>) () -> CookieHandler.getDefault());
> 
> Not sure if there is a policy for this, but you could replace the lambda with the method reference?

Good idea. Done.

> modules/javafx.web/src/main/java/com/sun/webkit/network/CookieJar.java line 71:
> 
>> 69:         @SuppressWarnings("removal")
>> 70:         CookieHandler handler =
>> 71:             AccessController.doPrivileged((PrivilegedAction<CookieHandler>) () -> CookieHandler.getDefault());
> 
> Same here

Done.

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

PR: https://git.openjdk.java.net/jfx/pull/681