Looks good. Thank.
Shura > On Jan 12, 2017, at 12:43 AM, Vikrant Agarwal <vikrant.v.agar...@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