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

Michael Strauß mstrauss at openjdk.org
Mon Jan 16 23:47:15 UTC 2023


On Fri, 13 Jan 2023 09:18:56 GMT, Florian Kirmaier <fkirmaier at openjdk.org> wrote:

>> 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

Marked as reviewed by mstrauss (Committer).

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

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


More information about the openjfx-dev mailing list