Skip to content

Commit

Permalink
8296590: StraightLineTest fails always on Linux and sometimes on othe…
Browse files Browse the repository at this point in the history
…r platforms

Reviewed-by: kcr
  • Loading branch information
Jay Bhaskar committed Apr 27, 2023
1 parent 0b36052 commit 1975165
Showing 1 changed file with 3 additions and 3 deletions.
Expand Up @@ -29,7 +29,6 @@
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;
import static org.junit.Assume.assumeFalse;

import java.util.concurrent.CountDownLatch;

Expand Down Expand Up @@ -107,8 +106,6 @@ public void setupTestObjects() {
}

@Test public void testLine() {
// JDK-8296590
assumeFalse(PlatformUtil.isLinux());

final CountDownLatch webViewStateLatch = new CountDownLatch(1);

Expand Down Expand Up @@ -137,6 +134,7 @@ public void setupTestObjects() {
"margin:0px;\n"+
"}\n" +
"div {\n" +
"white-space:nowrap;\n"+
"padding:0px;\n"+
"width:150px;\n"+
"height:20px;\n"+
Expand All @@ -154,6 +152,8 @@ public void setupTestObjects() {
});

assertTrue("Timeout when waiting for focus change ", Util.await(webViewStateLatch));
//introduce sleep , so that web contents would be loaded , then take snapshot for testing
Util.sleep(1000);

Util.runAndWait(() -> {
WritableImage snapshot = straightLineTestApp.primaryStage.getScene().snapshot(null);
Expand Down

1 comment on commit 1975165

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.