RFR: 8256821: TreeViewSkin/Behavior: misbehavior on switching skin [v3]

Ambarish Rapte 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).

-------------

PR: https://git.openjdk.java.net/jfx/pull/358


More information about the openjfx-dev mailing list