RFR: 8269081: Tree/ListViewSkin: must remove flow on dispose
Ambarish Rapte
arapte at openjdk.java.net
Thu Sep 2 09:20:40 UTC 2021
On Fri, 27 Aug 2021 11:15:58 GMT, Jeanette Winzenburg <fastegal at 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
More information about the openjfx-dev
mailing list