On Wed, 20 Sep 2023 21:12:55 GMT, Damon Nguyen <[email protected]> wrote:

> Converted test needs headful tag added to avoid failures.

Given the CI test failure, this is probably the right fix so that it can be 
integrated quickly.

As a follow-onto this, I see that PR #15802 deliberately tried to make this 
test headless by removing the unneeded `JFrame`. See [this 
comment](https://github.com/openjdk/jdk/pull/15802#discussion_r1329703581). 
What wasn't done is the removal of Robot, which is only being used for a sleep 
(and thus could be replaced with `Thread.sleep`). You might consider filing a 
low-priority follow-on bug to finish the job of making this test headless.

Ignore my comment. It crossed Phil's in the mail.

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

PR Comment: https://git.openjdk.org/jdk/pull/15847#issuecomment-1728454932
PR Comment: https://git.openjdk.org/jdk/pull/15847#issuecomment-1728457043

Reply via email to