On Fri, 20 Dec 2024 18:17:26 GMT, Harshitha Onkar <hon...@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
