<AWT Dev> Review request for JDK-8039279: Move first batch of functional tests to openjdk repository

Dmitriy Ermashov dmitriy.ermashov at oracle.com
Tue Apr 15 11:33:50 UTC 2014


Hi all.

Petr, thanks for review!

Guys, could you also review this tests?
http://cr.openjdk.java.net/~yan/8039279/webrev.02/

-- 
Thanks,
Dima


On 04/14/2014 03:41 PM, Petr Pchelko wrote:
> Looks fine to me.
>
> With best regards. Petr.
>
> On 14.04.2014, at 14:17, Dmitriy Ermashov <dmitriy.ermashov at oracle.com> wrote:
>
>> Hi,
>> Please, review the changeset for:
>> https://bugs.openjdk.java.net/browse/JDK-8039279
>>
>> Webrev is here:
>> http://cr.openjdk.java.net/~yan/8039279/webrev.02/
>>
>> Changelist:
>> 1. Test SetBounds was renamed to SetMaximizedBounds. All unnecessary code has been removed.
>> 2. Thrown away all usages of javax.swing package.
>> 3. Fixed info messages for Asserts functions.
>> 4. Code cleanup in LockingKeyStateTest (removed commented lines of code, switched to method reference usage)
>>
>> Test runs on the following platforms seems stable:
>> Windows 7 x64, Intel graphical card
>> Ubuntu Linux 12.04 x64, Intel graphical card
>> OS X 10.8.5 x64, NVIDIA GeForce 9400
>> Ubuntu 10.04 ARMv7
>>
>> -- 
>> Thanks,
>> Dima
>>



More information about the awt-dev mailing list