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

Petr Pchelko petr.pchelko at oracle.com
Fri Apr 18 06:38:41 UTC 2014


Hello, Dmitriy.

The fix looks good.

With best regards. Petr.

On Apr 17, 2014, at 1:22 PM, 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.03/
> 
> Changelist:
> 1. Removed internal API usage (sun.awt.*)
> 2. Removed Asserts library dependency from all tests except of ModifierRobotKeyTest (too much usage of asserts in it).
> 3. Improved thread synchronization in ComponentPreferredSize, ChangeGridSize, ModifierRobotKeyTest, LockingKeyStateTest
> 4. Added disposing frame at the end of each test.
> 
> 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
> 
> --Dima



More information about the awt-dev mailing list