RFR: 8246745: ListCell/Skin: misbehavior on switching skin
Jeanette Winzenburg
fastegal at openjdk.java.net
Fri Jun 26 10:46:19 UTC 2020
On Wed, 10 Jun 2020 23:33:46 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> ListCellSkin installs listeners to the ListView/fixedCellSize that introduce a memory leak, NPE on replacing the
>> listView and incorrect update of internal state (see bug report for details)
>> Fixed by removing the listeners (and the internal state had been copied from listView 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.
>
> @arapte can you review?
Hmm .. bottleneck seems to be layout as such (including css), accessing an instance field vs. querying a property
doesn't make a difference (at least none I could see).
-------------
PR: https://git.openjdk.java.net/jfx/pull/251
More information about the openjfx-dev
mailing list