On Fri, 15 Mar 2024 08:39:51 GMT, Renjith Kannath Pariyangad 
<rkannathp...@openjdk.org> 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 
> one additional commit since the last revision:
> 
>   Added summary

Changes requested by aivanov (Reviewer).

test/jdk/java/awt/print/PrinterJob/SecurityDialogTest.java line 48:

> 46:  * @run main/manual/othervm -Djava.security.manager=allow 
> SecurityDialogTest
> 47:  */
> 48: public class SecurityDialogTest extends Frame {

Suggestion:

public class SecurityDialogTest {

It does not need to extend `Frame`.

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

PR Review: https://git.openjdk.org/jdk/pull/18299#pullrequestreview-1938621163
PR Review Comment: https://git.openjdk.org/jdk/pull/18299#discussion_r1526078681

Reply via email to