On Wed, 4 Sep 2024 05:48:53 GMT, Tejesh R <t...@openjdk.org> wrote: >> Test failed intermittently on particular host. Though analysis pointed out >> to a test frame at left top on that host, I've updated the test for further >> stabilizing it. Two things done here: >> 1. Move the frame to center of the screen rather than left top. >> 2. Added tolerance checks for color comparison - this is based on analysis >> reports where the image didn't had exact black color which is supposed to >> be. So like other test cases, providing some tolerance for comparison. > > Tejesh R has updated the pull request incrementally with one additional > commit since the last revision: > > Updated based on review comments
Updated changes LGTM, I'm okay with color tolerance set to 10 but if a smaller value works for this test (in CI testing) then it might be better to be more restrictive with color tolerance. ------------- PR Comment: https://git.openjdk.org/jdk/pull/20723#issuecomment-2329586131
