On Thu, 20 Oct 2022 20:48:07 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
> Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and > a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > Make sure to configure the current test in LeakTest: > protected final Type WE_ARE_TESTING = Type.PAGINATION; > > Found another issue: Pagination class does not survive replacing its skin > (all components disappear). > > caused by: > - adding and not removing listeners > - adding and not removing children Nodes > - setting control's properties in the constructor > - incorrectly setting a clip rectangle > > NOTE: the fix will requires both ListenerHelper > [JDK-8294809](https://bugs.openjdk.org/browse/JDK-8294809) and Skin.install() > [JDK-8290844](https://bugs.openjdk.org/browse/JDK-8290844) changes. This pull request has now been integrated. Changeset: 0a785ae0 Author: Andy Goryachev <ango...@openjdk.org> URL: https://git.openjdk.org/jfx/commit/0a785ae036f48c736b65df865a3b93f954d46fe5 Stats: 128 lines in 2 files changed: 56 ins; 44 del; 28 mod 8295754: PaginationSkin: memory leak when changing skin Reviewed-by: kcr, aghaisas ------------- PR: https://git.openjdk.org/jfx/pull/925