Closed Bug 619997 Opened 11 years ago Closed 10 years ago

Kill Document.renameNode

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla6

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Details

(Keywords: dev-doc-complete, Whiteboard: [wdc])

Attachments

(1 file)

Patch v1
3.53 KB, patch
sicking
: review+
Details | Diff | Splinter Review 
Document.renameNode has never been implemented, and has been removed from Web DOM Core, we should remove it from the IDL as well. Patch coming up.
Attached patch Patch v1 — — Splinter Review 
As promised.
Attachment #518065 - Flags: review?(jonas)
Depends on: post2.0
Whiteboard: [wdc] → [wdc][needs landing]
Needs IID changes for things inheriting from nsIDOMDocument.
Whiteboard: [wdc][needs landing] → [wdc][needs landing][not-ready-for-cedar]
No longer depends on: post2.0
Whiteboard: [wdc][needs landing][not-ready-for-cedar] → [wdc][need gk2.2 ship]
http://hg.mozilla.org/mozilla-central/rev/4e6378990125

I don't think this needs an awful lot of docs, the only things that changed here is that document.renameNode === undefined and that document.renameNode() throws a different exception.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: dev-doc-needed
Resolution: --- → FIXED
Whiteboard: [wdc][need gk2.2 ship] → [wdc]
Target Milestone: --- → mozilla6
Documentation updated by removing it renameNode from the list of methods here:

https://developer.mozilla.org/en/DOM/document

I'm not mentioning this on Firefox 6 for developers since the method was never implemented anyway.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.