On Tue, 1 Aug 2023 20:14:17 GMT, Alexander Zvegintsev <[email protected]>
wrote:
>> 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.
It's *always* easier to run the test directly without jtreg.
It was in the original test case, you can dispose of a window by clicking. I
see no real problem preserving it. :)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14825#discussion_r1281130652
