<AWT Dev> [8u-backport] Review request for 8195738 : scroll poistion in ScrollPane is reset after calling validate()

Patrick Chen chen.j.patrick at gmail.com
Wed Mar 14 12:51:16 UTC 2018


Hi,
Why in src/windows/native/sun/windows/awt_ScrollPane.cpp did you delete
the main condition?

Means : the two variables are used isn't it ?

Cheers,
Pc


2018-03-14 11:02 GMT+01:00 Dipak Kumar <dipak.kumar at oracle.com>:

> Hi All,
>
>
>
> Please review the below patch (for 8u-backport) -
>
>
>
> Webrev : http://cr.openjdk.java.net/~dkumar/8195738/webrev.00/
>
> JBS : https://bugs.openjdk.java.net/browse/JDK-8195738
>
> JDK 11 changeset - http://hg.openjdk.java.net/jdk/client/rev/96bebffe0be1
>
>
>
> Patch pushed for JDK 11 applies cleanly to JDK 8 after adjusting file path
> except 'headful' keyword has been removed from the test as this is not
> defined for 8.
>
> Related Jtreg tests have been run against the proposed patch and results
> are fine.
>
>
>
> Thanks,
>
> Dipak
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20180314/71a7f75d/attachment.html>


More information about the awt-dev mailing list