On Tue, 28 Jan 2025 14:51:57 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

> The `javax/swing/JButton/4796987/bug4796987.java` test strictly verifies if 
> it's run on Windows XP, for this reason it never runs on modern versions of 
> Windows.
> 
> Since Windows XP is obsolete and visual styles cannot be disabled since 
> Windows 8, I removed the OS version check completely.
> 
> I captured an image of the panel and analyse colors in the image instead of 
> using `Robot.getPixelColor`; I save the image for reference if the test fails.
> 
> The updated test passes in the CI.

If the test is for only XP, then why to retain it ? Can't we remove the test 
since xp is not supported ?

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

PR Review: https://git.openjdk.org/jdk/pull/23336#pullrequestreview-2580488275

Reply via email to