RFR: 8294589: MenuBarSkin: memory leak when changing skin

[Jeanette Winzenburg]

On Fri, 30 Sep 2022 15:51:55 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

>> So here we have a problem - I know that the memory leak is fixed (I've tested using VisualVM in a real application), but without this change, or without adding a StageLoader, the test fails.
>> Any suggestions?
>
> hmm, no direct idea but ... sounds like it's not fixed completely: first step would be to track down why it fails for this particular skin and what exactly is different from all other skins (that are fixed so far). If it turns out that there's something special, you can do the special memory testing in SkinCleanupTest (with documentation why that's required).

btw: I would suggest that you believe the original test :) Added complexity might cancel out an existing problem while the underlying issue still persists in some way ..

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

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