<AWT Dev> JDK-8031423 : Test java/awt/dnd/DisposeFrameOnDragCrash/DisposeFrameOnDragTest.java fails by Timeout on Windows

[Yuri Nesterenko]

+1

-yan

On 04/15/2016 08:28 AM, Ajit Ghaisas wrote:
> Thanks Yuri.
>
> Here is the updated webrev with suggested change.
> http://cr.openjdk.java.net/~aghaisas/8031423/webrev.01/
>
> Regards,
> Ajit
>
>
> -----Original Message-----
> From: Yuri Nesterenko
> Sent: Thursday, April 14, 2016 6:48 PM
> To: Ajit Ghaisas; Sergey Bylokhov; awt-dev at openjdk.java.net
> Subject: Re: JDK-8031423 : Test java/awt/dnd/DisposeFrameOnDragCrash/DisposeFrameOnDragTest.java fails by Timeout on Windows
>
> Hi Ajit,
>
> it's good you didn't change java.awt.Robot!
> As to the test, you have created that instance of Robot:
> now, pass it to Util.waitForIdle() and not recreate every time.
>
> Thanks,
> -yan
>
> On 04/14/2016 02:59 PM, Ajit Ghaisas wrote:
>> Hi,
>>
>>       Adding delay in test code in main thread fixes this test timeout.
>>       I considered adding some delay in Robot.waitForIdle() method so that the calling thread yields, but realized that, it will have undesirable effect of slowing down test runs as many tests use that method.
>>
>>      This fix contains moving robot creation up and adding delay to the test.
>>      Request you to review webrev :
>>      http://cr.openjdk.java.net/~aghaisas/8031423/webrev.00/
>>
>> Regards,
>> Ajit
>>
>