On Tue, 24 Nov 2020 14:17:21 GMT, Jeanette Winzenburg <[email protected]> wrote:
> issues with behavior: > - memory leak due to an key eventHandler that's not removed > - after dispose, still modifying treeView (anchor) state due to listeners > selection that are not removed > > issues with skin: > - memory leak due to behavior leaking > - memory leak due to cellFactory in flow not removed > - throws NPE after switching (on modifying root children, refresh) due to > listeners not removed > > Fixed by cleaning up as needed. Added tests that are failing before and > passing after the fix. This pull request has now been integrated. Changeset: ca0d3d0f Author: Jeanette Winzenburg <[email protected]> URL: https://git.openjdk.java.net/jfx/commit/ca0d3d0f Stats: 158 lines in 6 files changed: 142 ins; 13 del; 3 mod 8256821: TreeViewSkin/Behavior: misbehavior on switching skin Reviewed-by: arapte ------------- PR: https://git.openjdk.java.net/jfx/pull/358
