RFR: 8296409: Multiple copies of accelerator change listeners are added to MenuItems, but only 1 is removed

[Dean Wookey]

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