<AWT Dev> Review request for JDK-8044765: Move functional tests AWT_SystemTray/Automated to openjdk repository

Petr Pchelko petr.pchelko at oracle.com
Wed Jun 4 11:08:29 UTC 2014


Hello, Dmitriy.

A couple of comments:
1. Could you please split loooong lines in ActionCommand.java, ModalityTest.java, MouseEventMaskTest.java, TrayIconEventModifiersTest.java, TrayIconEventsTest.java, TrayIconPopupTest.java
2. For interprocess communication we have special helpers in java/awt/regtesthelpers/process. You could reuse them to simplify InterJVM.java
3. MouseMovedTest - moved should be volatile

Thank you.
With best regards. Petr.

On 04 июня 2014 г., at 13:24, Dmitriy Ermashov <dmitriy.ermashov at oracle.com> wrote:

> Hi team,
> 
> It is new batch of functional tests, system tray tests this time.
> Please review the change set:
> http://cr.openjdk.java.net/~dermashov/8044765/webrev.00/
> 
> The fix also contains new test MouseMovedTest for verifying bug 7153700
> 
> -- 
> Thanks,
> Dima
> 



More information about the awt-dev mailing list