> Test was problemlisted due to it failing repeatedly on macos in automated > nightly testing citing "java.awt.IllegalComponentStateException: component > must be showing on the screen to determine its location " > Added delay after the setVisible() is called to make sure enough time is > given for frame to be visible, which is in sync with what we do for similar > other tests. > Also, made the frame to center of screen and increase the autoDelay time to > 100ms which is also same as done in similar to other tests. > Several iterations of the test works fine in all platforms after this > modification.
Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Copyright year fix ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3456/files - new: https://git.openjdk.java.net/jdk/pull/3456/files/e90b2616..d6541c03 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3456&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3456&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/3456.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3456/head:pull/3456 PR: https://git.openjdk.java.net/jdk/pull/3456