When menu buttons are added and removed from the scene, an accelerator change listener is added to each menu item in the menu. There is nothing stopping the same change listener being added multiple times.
MenuButtonSkinBase calls the ControlAcceleratorSupport.addAcceleratorsIntoScene(getSkinnable().getItems(), getSkinnable()); method each time the button is added to the scene, but that method itself also registers a listener to call itself. Each time the button is added to the scene, the method is called at least twice. When it's removed from the scene, the remove accelerator method is also called twice, but only the first call removes a change listener attached to the accelerator because the first call removes the entry from the hashmap changeListenerMap. The second call finds nothing in the map, and doesn't remove the additional instance. This pull request just removes the redundant code in the MenuButtonSkinBase. ------------- Commit messages: - 8296409: Stop additional change listeners being added Changes: https://git.openjdk.org/jfx/pull/937/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=937&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8296409 Stats: 26 lines in 2 files changed: 16 ins; 9 del; 1 mod Patch: https://git.openjdk.org/jfx/pull/937.diff Fetch: git fetch https://git.openjdk.org/jfx pull/937/head:pull/937 PR: https://git.openjdk.org/jfx/pull/937
