RFR: 8327177: Wrong GlobalRef deleted

[Kevin Rushforth]

On Mon, 4 Mar 2024 11:07:07 GMT, Johan Vos <jvos at openjdk.org> wrote:

> Explicitly remove old global ref value from menu->jCallback.
> This avoids the possibility of that value (which can be reused for other refs) to be deleted twice.

This is a safe fix that is clearly the right thing to do.

I recommend changing the Bug title (and then the PR title to match) to add "macOS" and "GlassMenu" somewhere in the title to make it a little more informative, but I'll leave it up to you.

This is a simple enough fix that a single reviewer is sufficient. Once integrated, I recommend backporting this to jfx22u for 22.0.1 (and possibly to earlier releases as you see fit).

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

Marked as reviewed by kcr (Lead).

PR Review: https://git.openjdk.org/jfx/pull/1388#pullrequestreview-1914377783