On Tue, 1 Aug 2023 19:22:11 GMT, Alexey Ivanov <[email protected]> wrote:
>> Tejesh R has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Review fix
>
> test/jdk/java/awt/Multiscreen/MultiScreenCheckScreenIDTest.java line 133:
>
>> 131: public void mouseClicked(MouseEvent e) {
>> 132: ((Window) e.getSource()).dispose();
>> 133: }
>
> In the automatic test, it's not necessary at all — you don't expect any input.
Sometimes it's easier to run the test directly, without the jtreg.
Dispose is useful in this case to avoid locking the user's screen.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14825#discussion_r1281109105