RFR: 8311881: jdk/javax/swing/ProgressMonitor/ProgressTest.java does not show the ProgressMonitorInputStream all the time

Alexander Zvegintsev azvegint at openjdk.org
Wed Jul 12 10:23:56 UTC 2023


On Wed, 12 Jul 2023 07:36:56 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> 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



More information about the client-libs-dev mailing list