Emit /> instead of /> when closing SVG empty elements

For consistency with XMLSerializer, only void XHTML elements should have a space.
Ideally such change should also be applied on other XML documents, but trying here to keep things simple.

Author: PolariTOON

cjs/interface/element.js

@@ -459,7 +459,7 @@ class Element extends ParentNode {
           const start = next[START];
           if (isOpened) {
             if ('ownerSVGElement' in start)
-              out.push(' />');
+              out.push('/>');
             else if (isVoid(start))
               out.push(ignoreCase(start) ? '>' : ' />');
             else

esm/interface/element.js

@@ -461,7 +461,7 @@ export class Element extends ParentNode {
           const start = next[START];
           if (isOpened) {
             if ('ownerSVGElement' in start)
-              out.push(' />');
+              out.push('/>');
             else if (isVoid(start))
               out.push(ignoreCase(start) ? '>' : ' />');
             else

test/interface/element.js

@@ -37,7 +37,7 @@ assert(htmlDocWithEmptyAttrFromSet.firstChild.getAttribute('style'), '');
 assert(htmlDocWithEmptyAttrFromSet.firstChild.outerHTML, '<span></span>');
 assert(htmlDocWithEmptyAttrFromSet.innerHTML, '<span></span>');
 
-const xmlDoc = parser.parseFromString(`<hierarchy><android.view.View content-desc="text3&amp;more"/></hierarchy>`, 'text/xml').documentElement;
+const xmlDoc = parser.parseFromString(`<hierarchy><android.view.View content-desc="text3&amp;more" /></hierarchy>`, 'text/xml').documentElement;
 
 assert(xmlDoc.firstChild.getAttribute('content-desc'), 'text3&amp;more');
 assert(xmlDoc.firstChild.outerHTML, '<android.view.View content-desc="text3&amp;more" />');
@@ -48,7 +48,7 @@ assert(xmlDoc.firstChild.getAttribute('content-desc'), '');
 assert(xmlDoc.firstChild.outerHTML, '<android.view.View content-desc="" />');
 assert(xmlDoc.innerHTML, '<android.view.View content-desc="" />');
 
-const xmlDocWithEmptyAttrFromSet = parser.parseFromString(`<hierarchy><android.view.View style=""/></hierarchy>`, 'text/xml').documentElement;// attribute is in emptyAttributes set is empty (even for XML)
+const xmlDocWithEmptyAttrFromSet = parser.parseFromString(`<hierarchy><android.view.View style="" /></hierarchy>`, 'text/xml').documentElement;// attribute is in emptyAttributes set is empty (even for XML)
 assert(xmlDocWithEmptyAttrFromSet.firstChild.getAttribute('style'), '');
 assert(xmlDocWithEmptyAttrFromSet.firstChild.outerHTML, '<android.view.View style="" />');
 assert(xmlDocWithEmptyAttrFromSet.innerHTML, '<android.view.View style="" />');

test/shared/parse-json.js

@@ -35,7 +35,7 @@ let div = parseJSON('[1,"div",1,"svg",1,"rect",-2,1,"c-e",-2]');
 
 assert(JSON.stringify(toJSON(div)), '[1,"div",1,"svg",1,"rect",-2,1,"c-e",-2]');
 
-assert(div.toString(), '<div><svg><rect /></svg><c-e></c-e></div>');
+assert(div.toString(), '<div><svg><rect/></svg><c-e></c-e></div>');
 
 assert(div.querySelector('c-e').constructed, void 0, 'not constructed');
 

test/svg/element.js

@@ -16,8 +16,8 @@ assert(svg instanceof SVGElement, true, '<svg> is an instance of a facade');
 assert('ownerSVGElement' in svg, true, '<svg> ownerSVGElement');
 assert(svg.ownerSVGElement, null, '<svg> ownerSVGElement is null');
 assert(svg.firstChild.ownerSVGElement, svg, '<rect> has an ownerSVGElement');
-assert(document.toString(), '<div><svg><rect /></svg></div>', 'svg nodes are OK');
-assert(document.documentElement.cloneNode(true).outerHTML, '<div><svg><rect /></svg></div>', 'svg cloned');
+assert(document.toString(), '<div><svg><rect/></svg></div>', 'svg nodes are OK');
+assert(document.documentElement.cloneNode(true).outerHTML, '<div><svg><rect/></svg></div>', 'svg cloned');
 
 assert(JSON.stringify(document), '[9,1,"div",1,"svg",1,"rect",-4]');
 assert(JSON.stringify(svg), '[1,"svg",1,"rect",-2]');
@@ -39,11 +39,11 @@ assert(svg.className.what, 'ever', '<svg>.className');
 svg.setAttribute('test', 123);
 svg.setAttribute('style', 'width:100px');
 
-assert(svg.toString(), '<svg style="width:100px" test="123"><rect /></svg>');
+assert(svg.toString(), '<svg style="width:100px" test="123"><rect/></svg>');
 
 svg.className = 'a b c';
 assert(svg.getAttribute('class'), 'a b c');
 
 svg.setAttribute('class', 'd e');
 assert(svg.getAttribute('class'), 'd e');
-assert(svg.namespaceURI, 'http://www.w3.org/2000/svg');
+assert(svg.namespaceURI, 'http://www.w3.org/2000/svg');

test/worker.js

@@ -3,6 +3,6 @@ import createAssert from './assert-es.js';
 
 const assert = createAssert.for('Web Worker');
 
-let {document} = parseHTML('<div><svg><rect /></svg></div>');
+let {document} = parseHTML('<div><svg><rect/></svg></div>');
 
-assert(document.querySelector('div').firstChild.localName, 'svg', 'Should be an svg element in the div');
+assert(document.querySelector('div').firstChild.localName, 'svg', 'Should be an svg element in the div');

worker.js

@@ -8046,7 +8046,7 @@ let Element$1 = class Element extends ParentNode {
           const start = next[START];
           if (isOpened) {
             if ('ownerSVGElement' in start)
-              out.push(' />');
+              out.push('/>');
             else if (isVoid(start))
               out.push(ignoreCase(start) ? '>' : ' />');
             else