<Swing Dev> Review request for JDK-8172489

[Alexandre (Shura) Iline]

Looks good.

Thank.

Shura

> On Jan 12, 2017, at 12:43 AM, Vikrant Agarwal <vikrant.v.agarwal at oracle.com> wrote:
> 
> Hi All,
>  
> Please review the following:
>  
> Bug : https://bugs.openjdk.java.net/browse/JDK-8172489 <https://bugs.openjdk.java.net/browse/JDK-8172489>
>  
> Webrev Link: http://cr.openjdk.java.net/~vagarwal/8172489/webrev_01/ <http://cr.openjdk.java.net/~vagarwal/8172489/webrev_01/>
>  
> Summary: This is a fix for a bug in Jemmy, that we found while automating the SwingSet Demo.
> JSlider was not able to scroll to negative value on Mac because in JSliderAPIDriver.jump(final ComponentOperator oper, final ScrollAdjuster adj) 
> , newValue = -1; was being done, and after assigning a new value to newValue conditionally, it was calling JSliderAPIDriver. setValue(ComponentOperator oper, int value) 
> which was setting the new value using JSliderOperator.setValue only if the newValue is not -1, which had been set as initial value, this was being done as a check that the value of newValue has not changed, but when the slider needs to goto a negative value, then this gets stuck at 0, because -1 value cannot be assigned.
>  
> Please see the bug for more description.
>  
> Best Regards,
> Vikrant

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20170112/44f40def/attachment.html>