RFR: 8319779: SystemMenu: memory leak due to listener never being removed [v17]
Johan Vos
jvos at openjdk.org
Thu Jun 13 19:54:46 UTC 2024
> A listener was added but never removed.
> This patch removes the listener when the menu it links to is cleared. Fix for https://bugs.openjdk.org/browse/JDK-8319779
Johan Vos has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 18 commits:
- Merge branch 'master' into 8319779-systemmenu
- Add more type info
- Add type info
Fix issue where notifications are missed when a new property is used.
- Merge branch 'master' into 8319779-systemmenu
- Remove whiteline and language error
- Ignore test in case the underlying GlassSystemMenu is not supported.
- Update modules/javafx.graphics/src/main/java/com/sun/javafx/tk/quantum/GlassSystemMenu.java
Co-authored-by: John Hendrikx <john.hendrikx+github at gmail.com>
- Merge branch 'master' into 8319779-systemmenu
- Add additional test for IOOBE detection.
This test comes from JDK-8323787
- Revert some of the conditional bindings.
Clear menu construction when an menuitem that is a menu needs to be removed
Add a test for the latter
- ... and 8 more: https://git.openjdk.org/jfx/compare/762f5902...5c733838
-------------
Changes: https://git.openjdk.org/jfx/pull/1283/files
Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1283&range=16
Stats: 663 lines in 5 files changed: 462 ins; 192 del; 9 mod
Patch: https://git.openjdk.org/jfx/pull/1283.diff
Fetch: git fetch https://git.openjdk.org/jfx.git pull/1283/head:pull/1283
PR: https://git.openjdk.org/jfx/pull/1283
More information about the openjfx-dev
mailing list