On Tue, 27 May 2025 23:51:05 GMT, Harshitha Onkar <hon...@openjdk.org> wrote:

>> @mrserb 
>> 
>> I can remove the EDT but I do think retaining them will stabilize this test 
>> based on the context of test failure.
>> 
>> - Test does NOT fail when it is run multiple times on macos-aarch64 but only 
>> when entire test suite is run. Can it still be synchronization issue if it 
>> is not reproducible when test is run multiple times?   
>> 
>> - The test repeated fails on macos-aarch64 only when the entire test suite 
>> is run so I suspect this could be the effect of couple of Graphics tests 
>> that was recently open-sourced and indeed this matches with the timeline 
>> that this test started to fail repeatedly on macos-aarch64.
>> 
>> - I didn't see any recent source code changesets that has direct relation to 
>> this test failure. There was one recent change to copyArea and ongoing 
>> subsequent change https://github.com/openjdk/jdk/pull/25340 but the test 
>> does not hit the code path in Blit.c or MaskBlit.c so that eliminates these 
>> changes as the cause.
>
> At the very least this test requires to be de-problemlisted to obtain the 
> screenshots at the time of failure to see the state of the Frame.
> 
> This test repeatedly fails due to the same error at the same point (at x=267 
> y=212) which is about mid-way of the frame where the color is expected to be 
> black but the actual is white.
> 
> 
> java.lang.RuntimeException: Test failed for background region at x=267 y=212 
> (expected=ff000000 actual=ffffffff)
> at CopyAreaOOB.testRegion(CopyAreaOOB.java:97)
> at CopyAreaOOB.paint(CopyAreaOOB.java:77) 
> 
> 
> **Expected Frame**
> 
> <img 
> src="https://github.com/user-attachments/assets/b0af326e-f99f-474b-81a8-bb110cb0a37c";
>  alt="background" width="150" height="150">

So it does not seems to be a test bug?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25279#discussion_r2110640053

Reply via email to