<AWT Dev> Request for Review: 7170655 frame size change does not follow label font change
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Wed May 23 05:34:05 PDT 2012
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
>>>
>>
>>
>
--
Best regards, Sergey.
More information about the awt-dev
mailing list