9777 - Text::splitText() doesn't work when given Text::length

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect, P2)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

If you call textNode.splitText(textNode.length), it should leave the original
text node unchanged and should return an empty text node.  Right now it is
stopping JS execution (although it ought to throw an exception when
out-of-bounds):

http://www.fas.harvard.edu/~dbaron/dom/test/one-core-xml/Text

Comment 1

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Marking for beta, just because.

Comment 2

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

HTML DOM bugs are M11/P2 for Vidur.

Comment 3

[waqar]

SplitText is returning the NS_DOM_INDEX_SIZE_ERR. This is the correct thing to
per DOM spec.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Reopening.  Now that exceptions are working, it should return that when the
argument is *greater than* length, but not when it is equal to length.  That was
the point of the bug.  To quote from the DOM1 spec:

INDEX_SIZE_ERR: Raised if the specified offset is negative or greater than the
number of characters in data.

It's just as reasonable to pass Text::length as it is to pass 0 - both should
create one empty text node and one text node with all the text.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

In line 1 of previous comment, s/return that/throw INDEX_SIZE_ERR/

Comment 6

[waqar]

The fix is checked in and should be working in the next release.

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

*** Bug 16569 has been marked as a duplicate of this bug. ***

Comment 8

[ckritzer (gone)]

dbaron, could you regress this bug?  The URL for this bug appears to no longer
be available.  I'll close this out on Wednesday unless I hear otherwise from
you.  Thanks!

Comment 9

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Verified fixed, Linux apprunner 1999-10-25-08-M11, including the bug marked as a
duplicate.  The URL seems fine to me.