On Thu, 11 May 2023 12:00:59 GMT, Karthik P K <[email protected]> wrote:

>> The test was failing on the first run and then it was not failing. Cause for 
>> the failure looked to be the UI state not getting updated. Hence added call 
>> to `Toolkit` `firePulse` method.
>> 
>> Able to run the test consistently without failure in Linux after the fix.
>
> Karthik P K has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add latch for tooltip hide and mouse move events

Since `consumeAutoHidingEvents` property value is asserted in the test, I'm 
suspecting that it is getting asserted before the mouse moves away from the 
slider and `consumeAutoHidingEvents` property value is reset to original value.
Since the test is not failing consistently for me in Ubuntu, I believe 
possibility of product issue is less. Hence  added a latch and small delay to 
make sure that the property value is asserted only after the mouse is moved.
@kevinrushforth  could you please check this fix.
I have ran the test in Ubuntu 22.04 VM multiple times and test didn't fail.

-------------

PR Comment: https://git.openjdk.org/jfx/pull/1119#issuecomment-1543875208

Reply via email to