Fix missing SVG removeNode on IE (part of issue #5)

microsoft · Apr 18, 2016 · d9f2b26 · d9f2b26
1 parent 868367c
commit d9f2b26
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/jsanity-0.3.js b/jsanity-0.3.js
@@ -948,10 +948,16 @@ jSanity = {};
                 // Need to avoid DOM squatting, eg: test<form><input name=parentNode>  (Credit: Gareth Heyes)
                 // Use removeNode() on IE and remove() elsewhere
                 //  removeNode() doesn't exist on Chrome, remove() doesn't exist on IE
+                // Use removeChild if all else fails.  Example where this is necessary (IE11): 
+                //  <svg xmlns="http://www.w3.org/2000/svg"><g onload="javascript:alert(1)"></g></svg>
                 try {
                     nodesToRemove[ i ].removeNode( true );
                 } catch ( e ) {
-                    nodesToRemove[ i ].remove();
+                    try {
+                        nodesToRemove[ i ].remove();
+                    } catch ( e ) {
+                        nodesToRemove[ i ].parentNode.removeChild(nodesToRemove[ i ]);
+                    }
                 }
             }