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