On Wed, 12 Jul 2023 07:36:56 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

> Due to some threading issues, Progress Monitor is not shown in macos..
> Seems like we need to call ProgressMonitor swing component in EDT..

Marked as reviewed by azvegint (Reviewer).

test/jdk/javax/swing/ProgressMonitor/ProgressTest.java line 81:

> 79:                     if ((total % (1024*1024)) == 0) {
> 80:                         try {
> 81:                             EventQueue.invokeAndWait(() -> {

I think we could use `invokeLater` here for slightly faster test execution.

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

PR Review: https://git.openjdk.org/jdk/pull/14844#pullrequestreview-1526018891
PR Review Comment: https://git.openjdk.org/jdk/pull/14844#discussion_r1260958300

Reply via email to