On Mon, 25 Mar 2024 19:53:54 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: > > Fix spelling @DamonGuy The test in https://github.com/openjdk/jdk/pull/18362 and the one in this PR test similar functionality -Dialog Modality when using Print functions. Will combining or retaining only one test make more sense instead of 2 separate tests? test/jdk/java/awt/print/PrinterJob/PrinterDialogsModalityTest.java line 92: > 90: frame.add(print); > 91: frame.pack(); > 92: frame.setVisible(true); frame.setVisible(true); not required when using PassFailJFrame. ------------- PR Comment: https://git.openjdk.org/jdk/pull/18461#issuecomment-2019049516 PR Review Comment: https://git.openjdk.org/jdk/pull/18461#discussion_r1538329544