Shrink SVG paint-order to take less bits

LayoutTests/svg/css/script-tests/svg-paint-order.js

 description("Test paint-order.")
 
 if (window.testRunner)
     testRunner.dumpAsText();
 createSVGTestCase();
 
 var text = createSVGElement("text");
 text.setAttribute("id", "text");
 text.setAttribute("x", "100px");
 text.setAttribute("y", "100px");
@@ skipping 12 matching lines @@
     // Reset paint-order.
     text.removeAttribute("paint-order");
 
     // Run test
     text.setAttribute("paint-order", valueString);
     shouldBeEqualToString("getComputedStyle(text).paintOrder", expectedValue);
 }
 
 debug("");
 debug("Test pre-normalized correct variants of 'paint-order'");
-test("fill stroke markers", "fill stroke markers");
-test("fill markers stroke", "fill markers stroke");
-test("stroke fill markers", "stroke fill markers");
-test("stroke markers fill", "stroke markers fill");
-test("markers stroke fill", "markers stroke fill");
-test("markers fill stroke", "markers fill stroke");
+test("fill stroke markers", "fill");

[pdr., 2015/07/29 05:20:50]

Is this change correct?

This seems to make us match both Gecko and WebKit but the spec seems to say we
should return the computed value "as specified":
https://svgwg.org/svg2-draft/single-page.html#painting-PaintOrder

I made a little demo to test this with: http://jsfiddle.net/Lpxsuto8

I think it makes sense to change our behavior, just want to be sure to update
the spec if we do.

[Shanmuga Pandi, 2015/07/29 06:55:04]

 Is it ok to change the code as like old approach. 
For ex :
 test("fill stroke", "fill stroke markers") instead of  test("fill stroke",
"fill");  ? 
Even then we are not achieving the spec item , computed value "as specified".

+test("fill markers stroke", "fill markers");
+test("stroke fill markers", "stroke");
+test("stroke markers fill", "stroke markers");
+test("markers stroke fill", "markers stroke");
+test("markers fill stroke", "markers");
 
 debug("");
 debug("Test correct single keyword value of 'paint-order'");
-test("normal", "fill stroke markers");
-test("fill", "fill stroke markers");
-test("stroke", "stroke fill markers");
-test("markers", "markers fill stroke");
+test("normal", "normal");
+test("fill", "fill");
+test("stroke", "stroke");
+test("markers", "markers");
 
 debug("");
 debug("Test correct dual keyword values of 'paint-order'");
-test("fill stroke", "fill stroke markers");
-test("fill markers", "fill markers stroke");
-test("stroke fill", "stroke fill markers");
-test("stroke markers", "stroke markers fill");
-test("markers fill", "markers fill stroke");
-test("markers stroke", "markers stroke fill");
+test("fill stroke", "fill");
+test("fill markers", "fill markers");
+test("stroke fill", "stroke");
+test("stroke markers", "stroke markers");
+test("markers fill", "markers");
+test("markers stroke", "markers stroke");
 
 debug("");
 debug("Test invalid values of 'paint-order'");
-test("foo", "fill stroke markers");
-test("fill foo", "fill stroke markers");
-test("stroke foo", "fill stroke markers");
-test("markers foo", "fill stroke markers");
-test("normal foo", "fill stroke markers");
-test("fill markers stroke foo", "fill stroke markers");
+test("foo", "normal");
+test("fill foo", "normal");
+test("stroke foo", "normal");
+test("markers foo", "normal");
+test("normal foo", "normal");
+test("fill markers stroke foo", "normal");
 
 debug("");
 debug("Test pre-normalized correct variants of 'paint-order' (presentation attribute)");
-test_attr("fill stroke markers", "fill stroke markers");
-test_attr("fill markers stroke", "fill markers stroke");
-test_attr("stroke fill markers", "stroke fill markers");
-test_attr("stroke markers fill", "stroke markers fill");
-test_attr("markers stroke fill", "markers stroke fill");
-test_attr("markers fill stroke", "markers fill stroke");
+test_attr("fill stroke markers", "fill");
+test_attr("fill markers stroke", "fill markers");
+test_attr("stroke fill markers", "stroke");
+test_attr("stroke markers fill", "stroke markers");
+test_attr("markers stroke fill", "markers stroke");
+test_attr("markers fill stroke", "markers");
 
 debug("");
 debug("Test correct single keyword value of 'paint-order' (presentation attribute)");
-test_attr("normal", "fill stroke markers");
-test_attr("fill", "fill stroke markers");
-test_attr("stroke", "stroke fill markers");
-test_attr("markers", "markers fill stroke");
+test_attr("normal", "normal");
+test_attr("fill", "fill");
+test_attr("stroke", "stroke");
+test_attr("markers", "markers");
 
 debug("");
 debug("Test correct dual keyword values of 'paint-order' (presentation attribute)");
-test_attr("fill stroke", "fill stroke markers");
-test_attr("fill markers", "fill markers stroke");
-test_attr("stroke fill", "stroke fill markers");
-test_attr("stroke markers", "stroke markers fill");
-test_attr("markers fill", "markers fill stroke");
-test_attr("markers stroke", "markers stroke fill");
+test_attr("fill stroke", "fill");
+test_attr("fill markers", "fill markers");
+test_attr("stroke fill", "stroke");
+test_attr("stroke markers", "stroke markers");
+test_attr("markers fill", "markers");
+test_attr("markers stroke", "markers stroke");
 
 debug("");
 debug("Test invalid values of 'paint-order' (presentation attribute)");
-test_attr("foo", "fill stroke markers");
-test_attr("fill foo", "fill stroke markers");
-test_attr("stroke foo", "fill stroke markers");
-test_attr("markers foo", "fill stroke markers");
-test_attr("normal foo", "fill stroke markers");
-test_attr("fill markers stroke foo", "fill stroke markers");
+test_attr("foo", "normal");
+test_attr("fill foo", "normal");
+test_attr("stroke foo", "normal");
+test_attr("markers foo", "normal");
+test_attr("normal foo", "normal");
+test_attr("fill markers stroke foo", "normal");
 
 var successfullyParsed = true;
 
 completeTest();