RFR: 8299423: JavaFX Mac system menubar leaks [v5]

Florian Kirmaier fkirmaier at openjdk.org
Fri Jan 13 09:18:56 UTC 2023


> This PR fixes the leak in the mac system menu bar.
> 
> Inside the native code, NewGlobalRef is called for the callable.
> Which makes it into a "GC-Root" until DeleteGlobalRef is called.
> 
> The DeleteGlobalRef is never called for the MenuEntry, if it's removed from the menu without removing it's callable.
> This PR adds logic, whether the Menu is inserted. If it's not inserted in a Menu anymore, then DeleteGlobalRef is called, by calling `_setCallback` with the callable "null".
> 
> The unit test verifies, that this bug happened without this change, but no longer happens with this change.

Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision:

  JDK-8299423
  we now use the junit5 api

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/987/files
  - new: https://git.openjdk.org/jfx/pull/987/files/d1e26d47..5368b802

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=987&range=04
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=987&range=03-04

  Stats: 5 lines in 1 file changed: 2 ins; 0 del; 3 mod
  Patch: https://git.openjdk.org/jfx/pull/987.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/987/head:pull/987

PR: https://git.openjdk.org/jfx/pull/987


More information about the openjfx-dev mailing list