On Fri, 15 Mar 2024 12:13:52 GMT, Renjith Kannath Pariyangad
<[email protected]> wrote:
>> Hi Reviewers,
>>
>> I have updated the test with PassFailJFrame with information panel, earlier
>> this was informed through terminal.
>> Please review and let me know your suggestions if any.
>
> Renjith Kannath Pariyangad has updated the pull request incrementally with
> two additional commits since the last revision:
>
> - Removed frame
> - Updated PassFailJFrame object access
Changes requested by aivanov (Reviewer).
test/jdk/java/awt/print/PrinterJob/SecurityDialogTest.java line 43:
> 41: * @key printer
> 42: * @summary tests native and cross-platform page and print dialog
> 43: * when security manager is installed
Verifies "Print to file" option is disable if reading/writing files is not
allowed by Security Manager.
I believe this type of summary explains better the purpose of the test, and the
purpose is to ensure "Print to file" is disabled in writing to files is not
allowed, according to the instructions of in the test.
test/jdk/java/awt/print/PrinterJob/SecurityDialogTest.java line 69:
> 67: }
> 68:
> 69: PassFailJFrame passFailJFrame = PassFailJFrame.builder()
Suggestion:
PassFailJFrame passFailJFrame = PassFailJFrame.builder()
Removed redundant double space.
-------------
PR Review: https://git.openjdk.org/jdk/pull/18299#pullrequestreview-1939223692
PR Review Comment: https://git.openjdk.org/jdk/pull/18299#discussion_r1526449041
PR Review Comment: https://git.openjdk.org/jdk/pull/18299#discussion_r1526450895
