On Mon, 1 May 2023 15:34:35 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

> > EventListenerLeak test was not touched, as it is a separate manual test app 
> > instead of a JUnit test.
> 
> In addition to the manual test, the `javafx.web` module also has an automated 
> [EventListenerLeakTest](https://github.com/openjdk/jfx/blob/fbee1d2201270e7b3beb054ecb9fdf65ae11c43d/modules/javafx.web/src/test/java/test/javafx/scene/web/EventListenerLeakTest.java)
>  that could benefit from using JMemoryBuddy, but it would not be a trivial 
> change, so it might be better to file a follow-on bug for that.

I actually missed that. I looked briefly into it and I agree it is a larger 
change, I will file a follow-up bug for it after this PR is merged.

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

PR Comment: https://git.openjdk.org/jfx/pull/1121#issuecomment-1531235468

Reply via email to