RFR: 8316627: JViewport Test headless failure

[Damon Nguyen]

On Wed, 20 Sep 2023 21:29:21 GMT, Phil Race <prr at openjdk.org> wrote:

> How did this get through ? Was it not tested before pushing ? And if a test needs "Robot" it surely expects a headful envt ? In fact I am going to say that the problem here is using Robot at all. You don't need to add headful, you just need to get rid of Robot which is just being used to do unnecessary Thread.sleep() calls.

I believe a review comment was suggested to change Thread.sleeps to robot delays after testing at some point. I did re-test what I thought were all the tests after moving them to open. @azvegint suggested the same change back to Thread.sleep. I have a current test ongoing for this change instead. Waiting for completion before updating the PR once more

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

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