On Fri, 29 Oct 2021 05:23:39 GMT, Phil Race <p...@openjdk.org> wrote:
> Broader ? Please expand since the change you suggest is only about the cursor This issue JDK-8276058 is broader than the cursor, related, this is why I would like to exclude it from it. > The test part of the fix has multiple stability improvements, and there's no > reason to throw that out. And moving the pixel capture position to a more > safe position is fine too. I just think that -10 isn't sa\fe enough. Sure, I have no comments about the changes related to stability, but there are suspicious cases like using tolerance. > Who mentioned aarch64 ? Are you referring to you proposed change. I don't see > the aarch64 connection anywhere FWIW the actual hardware on which this was > seen is (I think) a 2013 Mac Pro running 10.15.7 I have mentioned that a few comments above, on aarch64 the cursor is always present on the screenshot, unlike x64. both systems are "similar". BTW I have found that the built-in screencapture tool has the ability to take a screenshot with and w/o cursor, so probably there is a way to specify that as well via some public API. Or maybe it is so fast that the cursor flickering is invisible. ------------- PR: https://git.openjdk.java.net/jdk/pull/6140
