Hello, Alexander.

In doTest you are doing closeAll in the end. But if the test is failed and any 
assert throws an exception, closeAll will not be called and a frame could be 
left on the screen. I think it's better to call closeAll from a finally block.

With best regards. Petr.

On 18 июля 2014 г., at 16:59, alexander stepanov 
<alexander.v.stepa...@oracle.com> wrote:

> Hello,
> 
> Could you please review the fix for
> https://bugs.openjdk.java.net/browse/JDK-8049617
> 
> webrev:
> http://cr.openjdk.java.net/~avstepan/8049617/
> 
> This is the next portion of functional AWT tests prepared for migration to 
> OpenJDK repository.
> 
> The tests were checked on Ubuntu 14.04 Linux, Solaris 11, Mac OS X 10.8.5 and 
> Windows 7.
> 
> Thanks,
> Alexander

Reply via email to