Integrated: 8269081: Tree/ListViewSkin: must remove flow on dispose
Jeanette Winzenburg
fastegal at openjdk.java.net
Thu Sep 2 11:34:30 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
This pull request has now been integrated.
Changeset: e9315014
Author: Jeanette Winzenburg <fastegal at openjdk.org>
URL: https://git.openjdk.java.net/jfx/commit/e9315014f4accd97d757689d5ff212dd536a6e61
Stats: 90 lines in 3 files changed: 68 ins; 18 del; 4 mod
8269081: Tree/ListViewSkin: must remove flow on dispose
Reviewed-by: arapte
-------------
PR: https://git.openjdk.java.net/jfx/pull/612
More information about the openjfx-dev
mailing list