> This PR removes all remaining uses of `AccessController` and
> `AccessControlContext`, which represent the last remaining uses of the
> terminally deprecated security APIs except for those in the `/ios/` or
> `/android/` directories.
>
> With the removal of doPrivileged and the `if (System.getSecurityManager() !=
> null)` code paths, the ACC is no longer used, so can be completely
> eliminated. Along with this, I removed all unused imports of security-related
> APIs and all related `@SuppressWarnings("removal") annotations.
>
> ### Notes to reviewers
>
> * Most of the changes were straight-forward removals of methods and fields to
> save, retrieve and pass around the `AccessControlContext`.
> * The Toolkit class stores a collection of listeners in a `WeakHashMap` with
> the listener as the key (thus weakly held) and the ACC as the value. We no
> longer need or want the ACC, but I kept the use of `WeakHashMap` and changed
> the value type to `Object`, storing a singleton dummy object as the value for
> each entry. This minimizes the changes, while preserving the behavior of
> reclaiming the entries when they are garbage collected.
Kevin Rushforth has updated the pull request incrementally with one additional
commit since the last revision:
additional comments
-------------
Changes:
- all: https://git.openjdk.org/jfx/pull/1638/files
- new: https://git.openjdk.org/jfx/pull/1638/files/793a0aa5..f01faa1e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=1638&range=01
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=1638&range=00-01
Stats: 6 lines in 2 files changed: 4 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jfx/pull/1638.diff
Fetch: git fetch https://git.openjdk.org/jfx.git pull/1638/head:pull/1638
PR: https://git.openjdk.org/jfx/pull/1638
