]> source.dussan.org Git - vaadin-framework.git/commitdiff
Fix LegacyComponentThemeChangeTest on Chrome 47
authorHenri Sara <hesara@vaadin.com>
Tue, 15 Dec 2015 13:41:53 +0000 (15:41 +0200)
committerVaadin Code Review <review@vaadin.com>
Tue, 15 Dec 2015 14:16:05 +0000 (14:16 +0000)
The test uses an Embedded component with non-existent Flash content.
This does not work on Chrome 47 so this change disables that part
of the test on Chrome like it was already disabled on IE8.

Change-Id: Ie60123247ec45b2138202fccd73d794bf98e533e

uitest/src/com/vaadin/tests/themes/LegacyComponentThemeChangeTest.java

index 8062fdbf398071a1e250cc3ca476a39c69f3fdf5..1814aa99df6ef0ec1329fcceec39b60531ef4f97 100644 (file)
@@ -72,8 +72,9 @@ public class LegacyComponentThemeChangeTest extends MultiBrowserTest {
     }
 
     private void assertEmbeddedTheme(String theme) {
-        if (BrowserUtil.isIE8(getDesiredCapabilities())) {
-            // IE8 won't initialize the dummy flash properly
+        if (BrowserUtil.isIE8(getDesiredCapabilities())
+                || BrowserUtil.isChrome(getDesiredCapabilities())) {
+            // IE8 and Chrome 47 won't initialize the dummy flash properly
             return;
         }
         EmbeddedElement e = $(EmbeddedElement.class).first();