Closed Bug 43170 Opened 24 years ago Closed 24 years ago

wrong exception being thrown for node.insertBefore

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect

Tracking

()

Status:
VERIFIED INVALID

People

(Reporter: ckritzer, Assigned: jst)

References

(
URL
)

Details

(Whiteboard: [nsbeta3-] trivial fix) 

Overview: node.insertBefore() is throwing NOT_FOUND_ERR instead of 
NO_MODIFICATION_ALLOWED_ERR

Steps to Reproduce:
1) Launch 2000-06-19-xx-M17 commercial build
2) Load http://bubblegum/ngdriver/suites/dom1/cdex023.html

Actual Results:  NOT_FOUND_ERR thrown

Expected Results: NO_MODIFICATION_ALLOWED_ERR

Platforms Tested/Found on:
- MacOS9 2000-06-19-08-M17 Commercial
- Linux6 2000-06-19-20-M17 Commercial
- Win98  2000-06-19-21-M17 Commercial

Additional Comments: I'll add a simplified testcase as soon as I can.
Setting self as QAContact.
QA Contact: gerardok → ckritzer
What is "node" in the testcase? (I can't access the testcase).

Anyway, we should fix this for nsbeta3 if we're violating the DOM Level 1 spec.
Status: NEW → ASSIGNED
Keywords: nsbeta3
Target Milestone: --- → M18
correctness of DOM1 implementation.
Keywords: correctness
It is important that we return the correct error values so that content 
developed for DOM1 APIs does not start depending on incorrect values.  Marking 
nsbeta3+
Whiteboard: [nsbeta3+] trivial fix
Marking nsbeta3-.
Whiteboard: [nsbeta3+] trivial fix → [nsbeta3-] trivial fix
As jst pointed out to me (thanks dude), this bug is invalid...
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
Marking VERIFIED INVALID
Status: RESOLVED → VERIFIED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.