<AWT Dev> Request for Review: 7170655 frame size change does not follow label font change

[Jonathan Lu]

Hello Sergey,

I made one automatic jtreg test case and put it to following link along 
with the patch.
http://cr.openjdk.java.net/~luchsh/7170655_3/

Could you please take a look?

Thanks!
- Jonathan

On 05/23/2012 08:34 PM, Sergey Bylokhov wrote:
> Hi, Jonathan,
> Fix looks good. But I guess this bug was not caught by the regression 
> tests, so new test should be added(automatic test is better).
>
> 23.05.2012 14:15, Jonathan Lu wrote:
>> Hi Sergey,
>>
>> On 05/22/2012 10:23 PM, Sergey Bylokhov wrote:
>>> Hi, Jonathan,
>>> Looks like this bug is duplicate of 
>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7161865
>>> And the reason of the issue is that we post UpdateEvent in 
>>> target.repaint() in XLabelPeer.setFont(), but we should paint native 
>>> part of the component in place[1] and then post PaintEvent[2].
>>>
>>>  - To fix [1] we can replace target.repaint() to repaint() in 
>>> setFont(). Does it solve your issue?
>>
>> Yes, this resolves my problem, and here's the updated patch from your 
>> suggestion,
>> http://cr.openjdk.java.net/~luchsh/7170655_2/
>>
>> Thanks a lot!
>> - Jonathan
>>
>>>
>>> 22.05.2012 11:47, Jonathan Lu wrote:
>>>> Hi awt-dev,
>>>>
>>>> Here's a patch for bug 7170655, could anybody please help to take a 
>>>> look?
>>>> http://cr.openjdk.java.net/~luchsh/7170655/
>>>>
>>>> The problem is that painting event from EDT during painting 
>>>> handling does not get processed immediately, so leave a lag to the 
>>>> user. My solution here is to coalesce and dispatch the new paint 
>>>> event right after it was posted. This happens only to AWT components.
>>>>
>>>> Thanks!
>>>> -Jonathan
>>>>
>>>
>>>
>>
>
>