On Fri, 5 May 2023 13:40:39 GMT, Lukasz Kostyra <[email protected]> wrote:

>> Verified these changes on all platforms and saw no abnormalities in test 
>> execution.
>> 
>> This change is based on a preliminary patch done by @aghaisas , with some 
>> minor changes and the addition of one Leak-related web test. 
>> EventListenerLeak test was not touched, as it is a separate manual test app 
>> instead of a JUnit test.
>
> Lukasz Kostyra has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add an EDT no-op and sleep for a bit
>   
>   On some platforms Swing memory leak tests occasionally failed, this
>   might help to mitigate this problem.

Looks good.

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

Marked as reviewed by kcr (Lead).

PR Review: https://git.openjdk.org/jfx/pull/1121#pullrequestreview-1414849053

Reply via email to