Request for review: 7125657 [macosx] SpreadSheet demo has the broken display when clicking outside of the table
Anthony Petrov
anthony.petrov at oracle.com
Tue Feb 14 04:56:29 PST 2012
On 2/14/2012 4:53 PM, Sergey Bylokhov wrote:
> 14.02.2012 14:40, Anthony Petrov пишет:
>> Hi Sergey,
>>
>> The code change itself looks fine. However, I have a couple questions:
>>
>> 1. Why was this paintPeer() call needed in the updateComponent() in
>> the first place? I assume there was some reason, and I wonder whether
>> we break something by removing it.
>>
> I don't know the primary reason, but I am sure that it doesn't called on
> windows. Small testcase was attached to 8-mr in which behavior on
> windows and unix are different. Note that Apple jdk6 works in the same
> way as on windows.
> >
> > 2. Have you run (at least automatic) SQE/JCK component
> painting-related tests to ensure there's no regressions introduced with
> this fix?
> I haven't found any regression, but it doesn't mean that they aren't
> present.
Thanks for the clarification. The fix looks OK to me.
--
best regards,
Anthony
>>
>> --
>> best regards,
>> Anthony
>>
>> On 2/10/2012 1:20 AM, Sergey Bylokhov wrote:
>>> Hi Everyone,
>>> There is no reason to paint native part of the peer on Update event.
>>> Note that we paint it in the XToolkit but most of the code(and this
>>> demo) is working, because XPanel.paintPeer is noop.
>>>
>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7125657
>>> Webrev can be found at:
>>> http://cr.openjdk.java.net/~serb/7125657/webrev.00
>>>
>
>
More information about the macosx-port-dev
mailing list