RFR: 8256821: TreeViewSkin/Behavior: misbehavior on switching skin [v3]
arapte at openjdk.java.net
Mon Dec 7 13:03:17 UTC 2020
On Mon, 7 Dec 2020 10:46:32 GMT, Jeanette Winzenburg <fastegal at openjdk.org> 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.
> Jeanette Winzenburg has updated the pull request incrementally with one additional commit since the last revision:
> rename for consistency
Marked as reviewed by arapte (Reviewer).
More information about the openjfx-dev