On Wed, 11 Aug 2021 21:16:43 GMT, lawrence.andrews 
<github.com+87324768+lawrence-and...@openjdk.org> wrote:

>> 1) This testcase was throwing error saying Error: Not a test or directory 
>> containing tests: java/awt/print/PrinterJob/XparColor.java
>> 2) Added @test tag so that this testcase can be run
>> 3) Since Applet is deprecated and its not supported any more in the future 
>> release remove applet and make the testcase to run as main
>> 4) Since this is a manual testcase instruction frame or dailog was not 
>> visible just print dialog was visible so fixed it.
>> 5) When the testcase was run via main method two print dialog was shown one 
>> was from main method and one more was from constructor. Fixed to show just 
>> one print dialog.
>> 6) Since JDK 17 introduce metal rendering touching the java2d code thought 
>> this testcase will be useful to checking the rendering. 
>> 
>> @shurymury
>
> lawrence.andrews has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Fixed test instruction and execute the testcase on metal only for mac

test/jdk/java/awt/print/PrinterJob/XparColor.java line 134:

> 132:     private static void createAndShowTestDialog() {
> 133:         String testInstruction = "This testcase will be launched twice 
> on mac platform and once on non mac platform.\n" +
> 134:                 "On Mac once with default pipeline and once again with 
> Metal pipeline.\n" +

I think it will be better to rephrase this line with "This test will be run 
with default pipeline in all OS and additionally with metal pipeline only in 
macOS"

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

PR: https://git.openjdk.java.net/jdk/pull/4769

Reply via email to