RFR: 8306137: Open source several AWT ScrollPane related tests [v6]

Tejesh R tr at openjdk.org
Thu May 11 08:22:10 UTC 2023


On Fri, 5 May 2023 12:34:19 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Tejesh R has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Removed whitespace error
>
> test/jdk/java/awt/ScrollPane/ComponentScrollTest.java line 87:
> 
>> 85:             });
>> 86: 
>> 87:             Thread.sleep(5000);
> 
> Sleeping for 5 seconds could be too much.
> 
> It would be great to ensure the updated test reproduces the problem that was fixed by [JDK-4342129](https://bugs.openjdk.org/browse/JDK-4342129) in 1.3.0 and 1.4.0. Otherwise, these 5 seconds could be a waste of time.

I guess 1 second will be just fine. Updating it to 1 sec.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13621#discussion_r1190800258



More information about the client-libs-dev mailing list