Hello, Dmitriy. The fix looks good.
With best regards. Petr. On Apr 17, 2014, at 1:22 PM, Dmitriy Ermashov <[email protected]> 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
