On Mon, 25 Mar 2024 23:53:55 GMT, Damon Nguyen <dngu...@openjdk.org> wrote:

>> Convert 
>> java/awt/print/PrinterJob/PrinterDialogsModalityTest/PrinterDialogsModalityTest.html
>>  applet test to main using PassFailJFrame
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Move test tags. Remove setVisible

test/jdk/java/awt/print/PrinterJob/PrinterDialogsModalityTest.java line 35:

> 33:  * @bug 4784285 4785920
> 34:  * @key printer
> 35:  * @requires (os.family == "windows")

@DamonGuy Even if we retain both the tests, I believe this test can be extended 
to all platforms and `@requires` tag may not be required.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18461#discussion_r1538390954

Reply via email to