On Tue, 15 Apr 2025 15:36:52 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Good idea. As a (minor) suggestion, I might loop "5" times before calling it 
>> a failure.
>
>> A bigger question is why are we using `int` coordinates? Wouldn't it make 
>> the test depend on the scale (and also on the screen resolution and window 
>> position)?
> 
> I run this frequently on my Windows laptop with 125% and have never seen a 
> problem. Unless we do, this seems unrelated to the bug being fixed.

I'd recommend waiting for a few short times (1, 3, 5ms seems good), and then an 
additional long time just in case the machine has experienced a hiccup at that 
very moment. There's no harm done in waiting a second for a test to fail, as 
that would not slow down tests in most cases.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1772#discussion_r2045029432

Reply via email to