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