RFR: 8328827: Convert java/awt/print/PrinterJob/PrinterDialogsModalityTest/PrinterDialogsModalityTest.html applet test to main [v2]

Damon Nguyen dnguyen at openjdk.org
Mon Mar 25 19:53:54 UTC 2024


On Sat, 23 Mar 2024 23:38:51 GMT, Phil Race <prr at openjdk.org> wrote:

>> Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix spelling
>
> test/jdk/java/awt/print/PrinterJob/PrinterDialogsModalityTest.java line 49:
> 
>> 47:              with two buttons: "Page Dialog" and "Print Dialog".
>> 48:              1. Make the "Test Frame" active by clicking on title.
>> 49:              2. Press "Page Dialog" button and a page dailog should popup.
> 
> dailog -> dialog

Fixed. Thanks.

> test/jdk/java/awt/print/PrinterJob/PrinterDialogsModalityTest.java line 68:
> 
>> 66:                 .title("PrinterDialogsModalityTest Test Instructions")
>> 67:                 .instructions(INSTRUCTIONS)
>> 68:                 .rows((int) INSTRUCTIONS.lines().count() + 2)
> 
> Oh, that's a nice idea. I'd been tweaking by hand.

I agree it's a nice idea, but I took this idea from other PR's I've reviewed. Not sure who started this first, but very handy.

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

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


More information about the client-libs-dev mailing list