On Fri, 30 Sep 2022 15:51:55 GMT, Jeanette Winzenburg <faste...@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
