[dom] Consider banning insertNode() of the Range's start node

Aryeh Gregor via GitHub <[email protected]> Tue, 11 Aug 2015 18:09:23 +0000

Newsgroups	gmane.comp.web.dom.general
Message-ID	<[email protected]>

ayg has just created a new issue for https://github.com/whatwg/dom:

== Consider banning insertNode() of the Range's start node ==
Test-case:

```html
<!doctype html>
foo
<script>
var range = document.creteRange();
range.setStart(document.body.firstChild, 1);
var s = "Body children before: " + document.body.childNodes.length;
try {
  range.insertNode(range.startContainer);
  document.body.textContent = s + ", after: " + 
document.body.childNodes.length;
} catch (e) {
  document.body.textContent = "Exception";
}
</script>
```

Firefox outputs "Body children before: 2, after: 3", which matches the
 spec: there is no special handling of this case, and it winds up 
happily splitting the text node, then removing the first text node and
 re-inserting it back where it was, leaving the first portion no 
longer selected.  Chrome throws.  IE is the same as Firefox, but if 
you change the offset from 1 to 0 it outputs "Body children before: 2,
 after: 2", so it probably just aborts silently in that case.

IE/Firefox's behavior is just silly, and Chrome's seems a bit harsh.  
I'm inclined to go with Chrome, out of the options available.  Failing
 that, I'd go with IE and at least special-case the scenario where we 
create an empty text node.  Firefox (which is the current spec) would 
be my last choice.

See https://github.com/whatwg/dom/issues/63