<AWT Dev> Review request for 6734341 : REGTEST fails: SelectionAutoscrollTest.html

[Vikrant Agarwal]

Hi Semyon,

Although the execution time of the previous version was shorter, but it was also failing many times, the extra execution time got introduced after repacing mouseMove() from java.awt.Robot with mouseMove() from test.java.awt.regtesthelpers.Util , this performes the mouse movement a little slower than the previous one, but also makes the test more stable, as the new version is passing on all the tested machines.

Best Regards,
Vikrant

-----Original Message-----
From: Semyon Sadetsky 
Sent: Thursday, March 03, 2016 5:53 PM
To: Vikrant Agarwal; Sergey Bylokhov; awt-dev at openjdk.java.net; Alexander Scherbatiy
Subject: Re: Review request for 6734341 : REGTEST fails: SelectionAutoscrollTest.html

Hi, Vikrant,

I ran the new test version on Windows. It's execution time is about 20 seconds while the previous version was instant.
Is that expected?

--Semyon



On 2/26/2016 3:53 PM, Vikrant Agarwal wrote:
> Hi Sergey,
>
> I have updated the test to a standalone test.
>
> Updated Webrev: 
> http://cr.openjdk.java.net/~ntv/vikrant/6734341/webrev.02/
>
> Bug: JDK-6734341 : REGTEST fails: SelectionAutoscrollTest.html
>
> Best Regards,
> Vikrant
>
> -----Original Message-----
> From: Sergey Bylokhov
> Sent: Thursday, February 25, 2016 2:03 AM
> To: Vikrant Agarwal; awt-dev at openjdk.java.net; Alexander Scherbatiy; 
> Semyon Sadetsky
> Subject: Re: Review request for 6734341 : REGTEST fails: 
> SelectionAutoscrollTest.html
>
> Hi, Vikrant.
> The @bug tag should be updated in the html file, because it have a @test tag. But I suggest to remove the html file and change SelectionAutoscrollTest.java to a standalone test(instead of Applet).
>
> On 24.02.16 11:25, Vikrant Agarwal wrote:
>> Hi All,
>>
>> Please review the fix for JDK9.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-6734341
>>
>> Webrev: http://cr.openjdk.java.net/~ntv/vikrant/6734341/webrev.00/
>>
>> Issue:  java/awt/TextArea/UsingWithMouse/SelectionAutoscrollTest  
>> test was always failing for mac and intermittently failing for other platforms.
>>
>> Fix: using mouseMove() from test.java.awt.regtesthelpers.Util insted 
>> of
>> mouseMove() from java.awt.Robot makes the test more stable and the 
>> test passes on all the tested machines including mac.
>>
>> Best Regards,
>>
>> Vikrant
>>
>
> --
> Best regards, Sergey.