RFR: JDK-8299052 : ViewportOverlapping test fails intermittently on Win10 & Win11 [v2]

Harshitha Onkar honkar at openjdk.org
Tue Jun 6 23:00:34 UTC 2023


On Tue, 6 Jun 2023 22:34:15 GMT, Phil Race <prr at openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review fix
>
> test/jdk/java/awt/Mixing/AWT_Mixing/OverlappingTestBase.java line 2:
> 
>> 1: /*
>> 2:  * Copyright (c) 2014, 2022, Oracle and/or its affiliates. All rights reserved.
> 
> 2022 ?

This was by miss :) 
Updated to 2023

> test/jdk/java/awt/Mixing/AWT_Mixing/ViewportOverlapping.java line 164:
> 
>> 162:         OverlappingTestBase.doMain(args);
>> 163:         captureScreen("Img_5.png");
>> 164:     }
> 
> It is just a test, so no need to over-engineer it but  it 'feels wrong' to specify the file name
> as foo.png here and separately inside captureScreen choose 'png' as the file type to use.
> Perhaps  captureScreen("Img_5.png", "png")  ?

Updated

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14289#discussion_r1220502248
PR Review Comment: https://git.openjdk.org/jdk/pull/14289#discussion_r1220502673



More information about the client-libs-dev mailing list