RFR: 8328370: Convert java/awt/print/Dialog/PrintApplet.java applet test to main [v2]
Damon Nguyen
dnguyen at openjdk.org
Wed Mar 20 16:26:39 UTC 2024
On Tue, 19 Mar 2024 23:04:54 GMT, Phil Race <prr at openjdk.org> wrote:
>> Damon Nguyen has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Remove unused frame
>> - splitUI addition. Add forceFail
>
> test/jdk/java/awt/print/Dialog/PrintModalDialog.java line 57:
>
>> 55: Finish printing or close the dialog. If both dialogs are modal,
>> 56: PASS the test. Otherwise, FAIL.
>> 57: """;
>
> I think we need more instructions - explaining Modal and what to expect.
> "Modal in this case means that it blocks the user from interacting with other windows in the same application
> You may still be able to interact with unrelated applications on the desktop.
> One sure way to test this is to first show the print dialog and then press "Fail", because if you can click on "Fail"
> and have it respond, then the print dialog was not modal. If clicking on it does nothing then cancel the print dialog
> and do the same for the other print dialog.
> If all is well, then press Pass."
Makes sense. Added modal description to instructions as suggested.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18362#discussion_r1532406097
More information about the client-libs-dev
mailing list