<AWT Dev> Review Request for 8160696: IllegalArgumentException: adding a component to a container on a different GraphicsDevice

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Fri Aug 12 13:04:48 UTC 2016


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; awt-dev at openjdk.java.net
>> 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.


More information about the awt-dev mailing list