Integrated: 8295531: ComboBoxBaseSkin: memory leak when changing skin

Andy Goryachev angorya at openjdk.org
Tue Dec 6 16:30:19 UTC 2022


On Tue, 18 Oct 2022 23:41:05 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

> as determined by SkinMemoryLeakTest (remove line 166) and a leak tester
> https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java
> 
> Affected skins:
> - ColorPicker
> - DatePicker
> - ComboBox
> 
> caused by:
> - out-of-order modification of the control property (skin.install)
> - adding skin nodes and not removing them in dispose()
> - adding listeners and not removing them in dispose()
> 
> NOTE: the fix will require not only ListenerHelper [JDK-8294809](https://bugs.openjdk.org/browse/JDK-8294809) but also Skin.install() [JDK-8290844](https://bugs.openjdk.org/browse/JDK-8290844) changes.

This pull request has now been integrated.

Changeset: bb139208
Author:    Andy Goryachev <angorya at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/bb139208222d7c045be0a3c6f07ef15edcb1625b
Stats:     185 lines in 6 files changed: 82 ins; 41 del; 62 mod

8295531: ComboBoxBaseSkin: memory leak when changing skin

Reviewed-by: aghaisas

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

PR: https://git.openjdk.org/jfx/pull/922


More information about the openjfx-dev mailing list