Integrated: 8274022 Additional Memory Leak in ControlAcceleratorSupport

[Florian Kirmaier]

On Sat, 30 Oct 2021 10:56:40 GMT, Florian Kirmaier <fkirmaier at openjdk.org> wrote:

> This fixes the new ControlAcceleratorBug which was Introduced in JavaFX17.
> To fix it, I've made the Value of the WeakHashMap also weak. 
> We only keep this value to remove it as a listener later on. Therefore there shouldn't be issues by making this value weak.
> 
> 
> I've seen this Bug very very often, in the last weeks. Most of the applications I've seen are somehow affected by this bug.
> 
> It basically **breaks every application with menu bars and multiple stages** - which is the majority of enterprise applications. It's especially annoying because it takes some time until the application gets unstable.
> 
> Therefore **I would recommend** after this fix is approved, **to make a new version for JavaFX17** with this fix because this bug is so severe.

This pull request has now been integrated.

Changeset: 0d5b8f8b
Author:    Florian Kirmaier <fkirmaier at openjdk.org>
Committer: Kevin Rushforth <kcr at openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/0d5b8f8b3f71b1301a65adcc41fe0c7308d4ff01
Stats:     46 lines in 2 files changed: 33 ins; 0 del; 13 mod

8274022: Additional Memory Leak in ControlAcceleratorSupport

Reviewed-by: mstrauss, kcr

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

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