On Fri, 19 May 2023 11:27:16 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org>
wrote:
>> test/jdk/javax/swing/JTableHeader/6889007/bug6889007.java line 87:
>>
>>> 85: for(int i = -shift; i < width + 2*shift; i++) {
>>> 86: robot.mouseMove(x++, y);
>>> 87: robot.delay(100);
>>
>> It looks like we don't need this delay since there is
>> `robot.setAutoDelay(100);` on line 56. A total delay of 200ms seems too much
>> to me.
>
> I dont think it's too much as compared to 500ms we have in some tests..Since
> I have tested in all platforms with this values without any problems
> observed, I will like to keep this value for now..
This seems OK to me for a rarely repeated operation.
Here it is repeated 405 times(at least for my testing), which leads to an
additional ~40.5 s of the test execution time.
100ms is still a pretty solid delay for mouse movement.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14026#discussion_r1198868406
