On Sun, 9 Aug 2020 17:11:27 GMT, Florian Kirmaier <fkirma...@openjdk.org> wrote:
>> tests/system/src/test/java/test/javafx/stage/FocusedWindowTestBase.java line
>> 60:
>>
>>> 59: static WeakReference<Stage> closedFocusedStageWeak = null;
>>> 60: static Stage closedFocusedStage = null;
>>> 61:
>>
>> These two can be instance fields (which is usually preferred for test
>> variables).
>
> They are called from the following static method:
> @BeforeClass
> public static void initFX() throws Exception {
> initFXBase();
> }
No, these two are only used from `testClosedFocusedStageLeakBase`, which is an
instance method. I removed `static`
while testing it and it compiles and runs fine.
-------------
PR: https://git.openjdk.java.net/jfx/pull/153
