58972 - Range insertNode not implemented

Status: VERIFIED FIXED

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

Description

[Dylan Schiemann]

BuildID:    20001031

insertNode DOM 2 Range method not yet implemented

Reproducible: Always
Steps to Reproduce:
1. attempt to insertNode in a Range returns not implemented error

Actual Results:  NS_ERROR_NOT_IMPLEMENTED

testcase to follow

Comment 1

[Dylan Schiemann]

Attachment: testcase

Comment 2

[Keyser Sose]

Reporter is this still a problem in the latest nightlies? It seems to work for
me in 2000120604. 

Comment 3

[Dylan Schiemann]

It is still in the 20001211 nightly... Anthony is working on DOM Range bugs.
You will see the error message in tools -> JavaScript console.

Comment 4

[Keyser Sose]

Marking NEW as per reporters comments.

Comment 5

[anthonyd]

setting milestone, and priority.
anthonyd

Comment 6

[Joe Francis]

finishing up range can be pushed off

Comment 7

[Jeff Yates]

I am attaching a JavaScript "Patch" that can be used as a work around for the 
range object.  This patch adds support for the following:
    Range.innerHTML - read only
    Range.extractContents - as per W3C specs
    Range.cloneContents - as per W3C specs
    Range.insertNode - as per W3C specs
    Range.surroundContents - as per W3C specs
    Range.deleteContents - as per W3C specs - fixes buggy support from Mozilla
    Range.jmyCompareNode - extends Mozilla's compareNode to include
                           the following 2 constants
    Range.NODE_BEFORE_AND_INSIDE = -1;
    Range.NODE_INSIDE_AND_AFTER = -2;

I would like a C++ programmer to implement these in C if possible.

Jeff Yates

Comment 8

[Jeff Yates]

Attachment: Patch for Range functionality

Comment 9

[Jeff Yates]

Attachment: Updated patch with test cases and code commenting (had an error in surroundContent method)

Comment 10

[anthonyd]

Jeff Yates,
It looks like you didnt specify the mime type for this patch you attached.  It 
just looks like garbage, can yuo reattach with mime type text please? Thanks.

anthonyd

Comment 11

[anthonyd]

movign to mozilla 0.9
anthonyd

Comment 12

[Jeff Yates]

Attachment: Patch js file

Comment 13

[Jeff Yates]

Attachment: Test Case

Comment 14

[rubydoo123]

moving to mozilla1.0

Comment 15

[anthonyd]

checked in

Comment 16

[anthonyd]

see 58969 for details

Comment 17

[rubydoo123]

catching up on verifications