RFR: 8294589: MenuBarSkin: memory leak when changing skin

[Andy Goryachev]

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

>> 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 ..

Perhaps the test is too artificial, something is not being done correctly or exactly as in the real application?  Using StageLoader or showControl() hooks up the missing dependencies.

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

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