diff --git a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java index 35adebff18..939de60bee 100644 --- a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java +++ b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java @@ -105,8 +105,7 @@ @FixMethodOrder(MethodSorters.NAME_ASCENDING) public class Test_org_eclipse_swt_browser_Browser extends Test_org_eclipse_swt_widgets_Composite { - // TODO Reduce to reasonable value - private static Duration MAXIMUM_BROWSER_CREATION_TIME = Duration.ofSeconds(90); + private static Duration MAXIMUM_BROWSER_CREATION_TIME = Duration.ofSeconds(10); static { try { @@ -114,7 +113,7 @@ public class Test_org_eclipse_swt_browser_Browser extends Test_org_eclipse_swt_w } catch (Exception e) { e.printStackTrace(); } - System.setProperty("org.eclipse.swt.internal.win32.Edge.timeout", Long.toString(MAXIMUM_BROWSER_CREATION_TIME.toMillis())); + System.setProperty("org.eclipse.swt.internal.win32.Edge.timeout", Long.toString(Duration.ofSeconds(90).toMillis())); } // CONFIG @@ -206,6 +205,7 @@ public void setUp() { } shell.setText(shellTitle); setWidget(browser); // For browser to occupy the whole shell, not just half of it. + shell.open(); testLog = new StringBuilder("\nTest log:\n"); if (SwtTestUtil.isGTK) { @@ -1430,6 +1430,7 @@ public void test_VisibilityWindowListener_eventSize() { @Override @Test public void test_isVisible() { + shell.setVisible(false); // Note. This test sometimes crashes with webkit1 because shell.setVisible() calls g_main_context_iteration(). See Bug 509411 // To reproduce, try running test suite 20 times in a loop. super.test_isVisible();