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