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

[Sergey Bylokhov]

How this test will fail before the fix? since the canvas is added only 
once "frms[0].add(canvas);" It seems that it should be added to the 
first and then to the second frame, no?

On 10.08.16 17:38, Mikhail Cherkasov wrote:
> Hi Sergey, all,
>
> There's an update test:
> http://cr.openjdk.java.net/~mcherkas/8160696/webrev.01/
>
> Thanks,
> Mikhail.
>
> On 05.08.2016 21:17, Sergey Bylokhov wrote:
>> Hi, Mikhail.
>> Please update the header of the test and rename it to some meaningful
>> name. Also note that the frames should be disposed at the end of the
>> test.
>>
>> On 05.08.16 19:19, Mikhail Cherkasov wrote:
>>> Hi all,
>>>
>>> Please review a fix for the following bug:
>>> https://bugs.openjdk.java.net/browse/JDK-8160696
>>> webrev:
>>> http://cr.openjdk.java.net/~mcherkas/8160696/webrev.00/
>>>
>>> Java throws exception: "IllegalArgumentException: adding a component to
>>> a container on a different GraphicsDevice" if you move a Java
>>> application to different monitor and
>>> add a Component to a Container on another graphics device, without
>>> explicitly
>>> removing the Component from its previous parent.
>>>
>>> Moving components between containers works fine if you remove the
>>> component
>>> manually before moving it to other container, however this is already
>>> done in addImpl
>>> method, so I've just put a removing the component from old parent before
>>> " checkGD".
>>> It works fine now and I'm not sure whether we need "checkGD" at all,
>>> however I leave it
>>> there just in case.
>>>
>>> Thanks,
>>> Mikhail.
>>
>>
>


-- 
Best regards, Sergey.