> 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 two additional commits since the last revision: - Revert unneeded master change - Revert unneeded master change ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3456/files - new: https://git.openjdk.java.net/jdk/pull/3456/files/2c6d9ed4..1ed8b214 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3456&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3456&range=02-03 Stats: 187 lines in 2 files changed: 36 ins; 120 del; 31 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
