Bug 514408 Test_OpenWindowListener_evaluateInCallback() opens IE window

Disabling test on Win32/OS X. On Win32 it interferes with other tests by
leaving a native window behind.

Change-Id: I586b4f940bedc6002792e9535c716127c168d4f8
Signed-off-by: Leo Ufimtsev <lufimtse@redhat.com>

1 files changed, 4 insertions, 1 deletions

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 276f368887..c3c244cf95 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
@@ -724,7 +724,10 @@ public void test_LocationListener_evaluateInCallback() {
 /** Verify that evaluation works inside an OpenWindowListener */
 @Test
 public void test_OpenWindowListener_evaluateInCallback() {
-	assumeTrue(!isWebkit1); // This works on Webkit1, but can sporadically fail, see Bug 509411
+	// This works on Webkit1, but can sporadically cause a vm crash, see Bug 509411.
+	// This technically works on OSX/Win32, but on Win32 this opens a native window that can conflict with other tests,
+	// where as on Gtk, it's closed when the parent is closed at the end of the test.
+	assumeTrue(isWebkit2);
 	AtomicBoolean eventFired = new AtomicBoolean(false);
 	browser.addOpenWindowListener(event -> {
 		browser.evaluate("SWTopenListener = true");