On Fri, 5 May 2023 12:34:19 GMT, Alexey Ivanov <aiva...@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

Reply via email to