Bypass css text (#2811)

* [DOM mocks] enable setting element.style

* Use the style setter directly

render/render.js

@@ -800,10 +800,10 @@ module.exports = function($window) {
 			// Styles are equivalent, do nothing.
 		} else if (style == null) {
 			// New style is missing, just clear it.
-			element.style.cssText = ""
+			element.style = ""
 		} else if (typeof style !== "object") {
 			// New style is a string, let engine deal with patching.
-			element.style.cssText = style
+			element.style = style
 		} else if (old == null || typeof old !== "object") {
 			// `old` is missing or a string, `style` is an object.
 			element.style.cssText = ""

test-utils/domMock.js

@@ -346,9 +346,8 @@ module.exports = function(options) {
 					get style() {
 						return style
 					},
-					set style(_){
+					set style(value){
-						// https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/style#Setting_style
+						this.style.cssText = value
-						throw new Error("setting element.style is not portable")
 					},
 					get className() {
 						return this.attributes["class"] ? this.attributes["class"].value : ""

test-utils/tests/test-domMock.js

@@ -665,16 +665,13 @@ o.spec("domMock", function() {
 			o(div.style.background).equals("url('/*foo*/')")
 
 		})
-		o("setting style throws", function () {
+		o("setting style updates style.cssText", function () {
 			var div = $document.createElement("div")
-			var err = false
+			div.style = "background: red;"
-			try {
+
-				div.style = ""
+			o(div.style.background).equals("red")
-			} catch (e) {
+			o(div.style.cssText).equals("background: red;")
-				err = e
-			}
 
-			o(err instanceof Error).equals(true)
 		})
 	})
 	o.spec("events", function() {