]> source.dussan.org Git - vaadin-framework.git/commitdiff
Re-apply old change which was excluded from master
authorArtur Signell <artur@vaadin.com>
Fri, 9 May 2014 10:29:25 +0000 (13:29 +0300)
committerVaadin Code Review <review@vaadin.com>
Fri, 9 May 2014 11:13:45 +0000 (11:13 +0000)
The merge script was faulty between May and June 2013 and merged changes
individually from both 7.0 and 7.1 to master. In the case that a commit from
7.0 was merged right before there was a "Merge: no" commit in 7.1, the changes
between the "Merge: no" commit and the previous merge point from 7.1 to master was
potentially never merged (the whole chain was merged as a no-op).

The only affected change seems to be

0a437a5 Revert "Replaced css inject hack in TestUtils with Page.Styles.add() #11798"
b008768 Implement parenthesis-handling fixes for Sass in Vaadin 7.1 (#12834)

The former one is re-applied in this commit, the latter one has been verified to be in
the master branch

Change-Id: If0cd75d75fdfc8893ac501aab2a0be1b55b09dcc

uitest/src/com/vaadin/tests/util/TestUtils.java

index dcd28c34136e7d10204d24b2c83586a0a102b207..5c6315a23aa191c44e1858092d8ddafe867069b5 100644 (file)
@@ -99,13 +99,22 @@ public class TestUtils {
             "YE", "ZAMBIA", "ZM", "ZIMBABWE", "ZW" };
 
     /**
-     * Injects css into the current window. Can be used to keep tests css in
-     * source files.
+     * Crossbrowser hack to dynamically add css current window. Can be used to
+     * keep tests css in source files.
      * 
      * @param cssString
      */
     public static void injectCSS(UI w, String cssString) {
-        w.getPage().getStyles().add(cssString);
+        String script = "if ('\\v'=='v') /* ie only */ {\n"
+                + "        document.createStyleSheet().cssText = '"
+                + cssString
+                + "';\n"
+                + "    } else {var tag = document.createElement('style'); tag.type = 'text/css';"
+                + " document.getElementsByTagName('head')[0].appendChild(tag);tag[ (typeof "
+                + "document.body.style.WebkitAppearance=='string') /* webkit only */ ? 'innerText' "
+                + ": 'innerHTML'] = '" + cssString + "';}";
+
+        w.getPage().getJavaScript().execute(script);
     }
 
     public static void installPerformanceReporting(TextArea targetTextArea) {