<AWT Dev> 8077409: Drawing deviates when validate() is invoked on java.awt.ScrollPane
Andrew Brygin
andrew.brygin at oracle.com
Wed Jun 3 10:25:09 UTC 2015
Hello Mikhail,
the change eliminates a discrepancy between the layout and the rendering,
and looks good.
A part of the layout related to the viewport size calculation looks a
bit misleading,
but it has no direct relation to the fix...
Thanks,
Andrew
On 5/25/2015 3:18 PM, mikhail cherkasov wrote:
> Hello,
>
> this is kindly reminder about the review request.
>
> Thanks,
> Mikhail.
>
> On 5/22/2015 7:28 PM, mikhail cherkasov wrote:
>> Hello Sergey.
>>
>> There's a second version of fix with a test:
>> http://cr.openjdk.java.net/~mcherkas/8077409v9/webrev.01/
>>
>> I found out that the problem is in layout method, it should not use
>> insets,
>> ViewportSize is already calculated with considering about insets.
>>
>> During testing jdk without the fix I faced with rendering problems
>> like described in jbs bug, also
>> I faced with duplication of very first pixel of scroll pane.
>>
>> With http://cr.openjdk.java.net/~mcherkas/8077409v9/webrev.01/
>> everything works fine. I don't see any rendering issues or problems
>> with scrolling.
>> I also run test from test/(closed)/java/awt/ScrollPane, all tests are
>> passed.
>>
>> Thanks,
>> Mikhail.
>>
>> On 5/5/2015 7:24 PM, Sergey Bylokhov wrote:
>>> Hi, Mikhail.
>>> Is it possible to write a test for this issue based on the code in
>>> the bug report?
>>>
>>> On 30.04.15 22:44, mikhail cherkasov wrote:
>>>> Hi there,
>>>>
>>>> Could you please review the fix:
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8077409
>>>> webrev: http://cr.openjdk.java.net/~mcherkas/8077409v9/webrev.00/
>>>>
>>>> In ScrollPane, move() is called without considering the quota of
>>>> Insets when invoking ScrollPane$PeerFixer.adjustmentValueChanged().
>>>> When
>>>> validate() is called, drawing deviates to coordinate with Insets
>>>> interanlly.
>>>>
>>>> Thanks,
>>>> Mikhail.
>>>>
>>>>
>>>
>>>
>>
>
More information about the awt-dev
mailing list