RFR: 8316627: JViewport Test headless failure

Kevin Rushforth kcr at openjdk.org
Wed Sep 20 21:36:42 UTC 2023


On Wed, 20 Sep 2023 21:12:55 GMT, Damon Nguyen <dnguyen at openjdk.org> 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


More information about the client-libs-dev mailing list