Bug 16867 - WebKit and FireFox disagree on complex Range.insertNode handling
: WebKit and FireFox disagree on complex Range.insertNode handling
Status: NEW
: WebKit
XML DOM
: 528+ (Nightly build)
: All Mac OS X 10.4
: P2 Normal
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2008-01-13 17:38 PST by
Modified: 2010-06-10 16:36 PST (History)


Attachments
Additional tests for Range.insertNode (8.99 KB, patch)
2009-02-16 11:24 PST, Eric Seidel
no flags Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2008-01-13 17:38:22 PST
WebKit and FireFox disagree on complex Range.insertNode handling

I just fixed bug 16765, at which time mitz pointed out that we might not handle all of the insertNode cases correctly yet.  This bug is to track the case where the node inserted is already the endContainer node, and the node insertion causes the startContainer to split (because it's a TEXT node).  Attaching test case shortly.
------- Comment #1 From 2009-02-16 02:52:25 PST -------
> Attaching test case shortly.

Any update on the test case?
------- Comment #2 From 2009-02-16 11:24:19 PST -------
Created an attachment (id=27702) [details]
Additional tests for Range.insertNode

 ...range-insertNode-already-in-range1-expected.txt |   21 ++++++++++++
 .../Range/range-insertNode-already-in-range1.html  |   13 +++++++
 ...range-insertNode-already-in-range2-expected.txt |   21 ++++++++++++
 .../Range/range-insertNode-already-in-range2.html  |   13 +++++++
 ...range-insertNode-already-in-range3-expected.txt |   22 ++++++++++++
 .../Range/range-insertNode-already-in-range3.html  |   13 +++++++
 .../range-insertNode-already-in-range1.js          |   34 +++++++++++++++++++
 .../range-insertNode-already-in-range2.js          |   34 +++++++++++++++++++
 .../range-insertNode-already-in-range3.js          |   35 ++++++++++++++++++++
 9 files changed, 206 insertions(+), 0 deletions(-)
------- Comment #3 From 2009-02-16 11:24:44 PST -------
Yay git!  I found the old branch and attached the commit with test cases!