On Fri, 27 Aug 2021 11:15:58 GMT, Jeanette Winzenburg <faste...@openjdk.org> 
wrote:

> left-over issue from cleanup of Tree/ListViewSkin: direct children that have 
> been added by the skin must be removed in dispose
> 
> fixed by removing the flow (which allowed to revert the previous cleanup of 
> event handlers/cellfactory)
> 
> added tests to verify
> - constant child count after replacing skin
> - no memory leak when switching skin while showing

Looks good to me.

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

Marked as reviewed by arapte (Reviewer).

PR: https://git.openjdk.java.net/jfx/pull/612

Reply via email to