On Mon, 2 Dec 2024 19:03:55 GMT, Harshitha Onkar <[email protected]> wrote:

>> The following two tests failed intermittently due to stability issues after 
>> JEP486 (Removal of Security Manager). Tests have been updated for stability. 
>> This test is applicable for linux platform only, hence `@requires (os.family 
>> == "linux")`  jtreg tag is added and the redundant OS check within the test 
>> is removed.
>> 
>> - java/awt/datatransfer/Independence/IndependenceAWTTest.java
>> - java/awt/datatransfer/Independence/IndependenceSwingTest.java
>> 
>> Multiple runs of the test on CI looks good and the tests pass.
>
> Harshitha Onkar has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   removed redundant null check

Tested the changes myself on linux and on CI with repeats. Looks stable to me 
and I didn't see anything odd. Updated changes also look fine. I suppose the 
modified waitForIdles and delays alongside moving things into the EDT did it.

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

Marked as reviewed by dnguyen (Committer).

PR Review: https://git.openjdk.org/jdk/pull/22490#pullrequestreview-2473724192

Reply via email to