Integrated: 8253634: TreeCell/Skin: misbehavior on switching skin

Jeanette Winzenburg fastegal at openjdk.java.net
Sun Nov 1 13:54:53 UTC 2020


On Fri, 25 Sep 2020 11:10:20 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

> TreeCellSkin installs listeners to the TreeView/fixedCellSize that introduce a memory leak, NPE on replacing the treeView and incorrect update of internal state.
> 
> Fixed by removing the listeners (and the internal state had been copied from treeView on change) and access of listView state when needed.
> 
> Added tests that failed before and pass after the fix, plus a sanity test to guarantee same (correct) behavior before/after.
> 
> Issue and fix is basically the same as for ListCellSkin [JDK-8246745](https://bugs.openjdk.java.net/browse/JDK-8246745)

This pull request has now been integrated.

Changeset: cb545cc6
Author:    Jeanette Winzenburg <fastegal at openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/cb545cc6
Stats:     101 lines in 4 files changed: 60 ins; 36 del; 5 mod

8253634: TreeCell/Skin: misbehavior on switching skin

Reviewed-by: arapte

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

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


More information about the openjfx-dev mailing list