looks fine to me.
On 11.08.16 19:57, Mikhail Cherkasov wrote:
Ambarish, thank you for comments, please review a new version:
http://cr.openjdk.java.net/~mcherkas/8160696/webrev.03/test/java/awt/Container/MoveToOtherScreenTest/MoveToOtherScreenTest.java.html
On 11.08.2016 9:07, Ambarish Rapte wrote:
Hi Mikhail,
Here are few suggestions,
Please add @key heedful jtreg tag.
Please correct line 74.
Use specific class imports instead of package import.
Regards,
Ambarish
-----Original Message-----
From: Mikhail Cherkasov
Sent: Wednesday, August 10, 2016 9:26 PM
To: Sergey Bylokhov; [email protected]
Subject: Re: <AWT Dev> Review Request for 8160696:
IllegalArgumentException: adding a component to a container on a
different GraphicsDevice
Now everything is in place:
http://cr.openjdk.java.net/~mcherkas/8160696/webrev.02/test/java/awt/Container/MoveToOtherScreenTest/MoveToOtherScreenTest.java.html
On 10.08.2016 18:32, Sergey Bylokhov wrote:
but these frames should be disposed
--
Best regards, Sergey.
