[Rev 01] RFR: 8241737: TabPaneSkin memory leak on replacing selectionModel

Jeanette Winzenburg fastegal at openjdk.java.net
Wed Apr 15 11:17:03 UTC 2020

On Wed, 15 Apr 2020 10:52:26 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

>> `TabPaneSkin` adds a listener to `SelectionModel.selectedItemProperty()` which holds the `SelectionModel` from being
>> GCed. Fix is to add and remove the listener when a `SelectionModel` is changed.
>> If the fix looks good, We can change all the `getSkinnable().getSelectionModel()` calls to use the new class member
>> `selectionModel`.
>> The fix seems safe to cause any regression. Enabled an ignored test that was added as part of fix for
>> [JDK-8241455](https://bugs.openjdk.java.net/browse/JDK-8241455).
> Ambarish Rapte has updated the pull request incrementally with one additional commit since the last revision:
>   Add tests

Marked as reviewed by fastegal (Author).


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

More information about the openjfx-dev mailing list