Re: RFR: 8208088: Memory Leak in ControlAcceleratorSupport [v5]

2021-04-12 Thread Ambarish Rapte
On Sat, 10 Apr 2021 14:58:14 GMT, Kevin Rushforth wrote: > I'd recommend to get rid of the indirect way of testing I have removed the indirect way, please take a look. - PR: https://git.openjdk.java.net/jfx/pull/429

Re: RFR: 8208088: Memory Leak in ControlAcceleratorSupport [v5]

2021-04-09 Thread Ambarish Rapte
> The method `ControlAcceleratorSupport.doAcceleratorInstall(final List extends MenuItem> items, final Scene scene)` adds a `ChangeListener` on > `MenuItem.acceleratorProperty()`. This listener is not removed when the > MenuItem is removed from scenegraph. > Adding and removing a MenuItem