RFR: 8242621: TabPane: Memory leak when switching skin [v4]
Kevin Rushforth
kcr at openjdk.java.net
Wed Jan 6 19:19:57 UTC 2021
On Wed, 6 Jan 2021 11:16:07 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:
>> `TabPaneSkin` installs some listeners that are not removed when `TabPaneSkin` is changed.
>> The fix converts listeners to WeakListeners and also removes them on dispose.
>>
>> There is a NPE check change needed in `isHosrizontal()`. Without this check it causes NPE if pulse is in progress while TabPaneSkin is getting disposed.
>>
>> `SkinMemoryLeakTest` already had a test which only needed to be enabled.
>> Test fails before and passes after this change.
>
> Ambarish Rapte has updated the pull request incrementally with two additional commits since the last revision:
>
> - Space corrections in tests
> - Review update: cleanup and corrections
Looks good.
-------------
Marked as reviewed by kcr (Lead).
PR: https://git.openjdk.java.net/jfx/pull/318
More information about the openjfx-dev
mailing list