<Swing Dev> [8] Review request for JDK-6757986: javax/swing/JInternalFrame/5066752/bug5066752.java needs correction

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Thu Dec 13 10:10:21 UTC 2012


    The fix looks good for me.

   Thanks,
   Alexandr.

On 12/11/2012 6:53 PM, vera akulova wrote:
> Hello Sergey,
>
> Thanks for your reply, I created new webrev: 
> http://cr.openjdk.java.net/~kshefov/6757986/webrev.01
> (typo near tag @test fixed, real coordinates are used to get color now)
>
> Thanks,
> Vera
>
> On 11.12.2012 16:11, Sergey Bylokhov wrote:
>> Hello, Vera.
>> setBounds() can be ignored by native system, I suggest to fetch real 
>> coordinates from the window instead of 50,50.
>> Also bug5066752 is a typo in 24   @test bug5066752 ?
>>
>> 10.12.2012 19:29, vera akulova wrote:
>>> Hello,
>>>
>>> Please review a fix for the issue:
>>>    JDK-6757986: javax/swing/JInternalFrame/5066752/bug5066752.java 
>>> needs correction
>>>
>>> The webrev is http://cr.openjdk.java.net/~kshefov/6757986/webrev.00/
>>>
>>> The manual test from closed repo was automated.  Fixed test works 
>>> fine on windows, linux, solaris and macos.
>>>
>>> Thanks,
>>> Vera.
>>
>>




More information about the swing-dev mailing list