> As indicated in the bug report, WebView needs to call to 
> `CookieManager::getDefault` within a `doPrivileged` block so that it will 
> work when a security manager is enabled. There are two calls in 
> `com.sun.webkit.network.CookieJar` that are not wrapped in a `doPrivileged`.
> 
> I added a manual test (since it requires loading a page over http/https which 
> we can't do in our automated tests) based on the test program that was 
> attached to JBS.

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

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

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/681/files
  - new: https://git.openjdk.java.net/jfx/pull/681/files/ca436714..1d7a6064

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jfx&pr=681&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=681&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jfx/pull/681.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/681/head:pull/681

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

Reply via email to