RFR: 8345538: Robot.mouseMove doesn't clamp bounds on macOS when trying to move mouse off screen [v3]

Alisen Chung achung at openjdk.org
Fri Dec 20 19:25:35 UTC 2024


On Fri, 20 Dec 2024 18:17:26 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:

>>> For this test you can add a null check before calling .getLocation() for sanity (although multimonitor setup is not the case for CI runs)
>> 
>> If `Robot.mouseMove` is expected to limit mouse to the valid (virtual screen) coordinates, `MouseInfo.getPointerInfo()` should never throw `NullPointerException` if mouse is available.
>
> @aivanov-jdk 
> I see your point. But I'm running into NPE at `MouseInfo.getPointerInfo().getLocation()` without the fix as well (dual monitor, extended display setup)
> 
> Is it expected of MouseInfo.getPointerInfo() to return null for off-screen coordinates (meaning does it consider it as mouse not available if it is outside the screen devices)?
> 
> PointerInfo has two things associated with it - screen device and a location, so if it is not able to associate the coordinate with any of the screen device (off-screen coordinate) is it suppose to return null in this case?

Interesting. Does this also happen after the fix with Robot clamping coordinates in shared code?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22781#discussion_r1894308586


More information about the client-libs-dev mailing list