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

Reply via email to