> When the Java layer removes a systemmenu, release the native resources > related to this systemmenu. > This removes the strong JNI Global ref, which prevents its references from > being gc'ed. > > The current implementation for the mac-specific system menu creates a menu, > but never releases its resources. In the `dealloc` of this menu, the strong > jni refs are deleted. > With this PR, we now release the native resources associated with a menuItem > when that one is removed from a menu. A consequence is that this menuItem > should never be used after being removed from its current menu (e.g. it > should not be re-added, or its text/shortcut should not be altered). > The current implementation will create a new MacMenuDelegate every time a > menuItem is inserted into a menu, so there should be no references to the > native resources lingering.
Johan Vos has updated the pull request incrementally with one additional commit since the last revision: Restore permissions on gradlew ------------- Changes: - all: https://git.openjdk.org/jfx/pull/1277/files - new: https://git.openjdk.org/jfx/pull/1277/files/5cf8c948..afd48a7e Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=1277&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1277&range=01-02 Stats: 0 lines in 1 file changed: 0 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jfx/pull/1277.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/1277/head:pull/1277 PR: https://git.openjdk.org/jfx/pull/1277
