Closed
Bug 581867
Opened 13 years ago
Closed 11 years ago
Remove nsIBaseDOMException
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: khuey, Assigned: Ms2ger)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
17.33 KB,
patch
|
peterv
:
review-
|
Details | Diff | Splinter Review |
This appears to have fallen out of disuse sometime before we left CVS. I didn't try too hard to track its origins down but the macros and the interface have been unused for a long time.
Attachment #460181 -
Flags: review?(jst)
Comment 1•13 years ago
|
||
Comment on attachment 460181 [details] [diff] [review] Patch Since peterv wrote this back in the day, I'll let him decide on its faith :)
Attachment #460181 -
Flags: review?(jst) → review?(peterv)
Comment 2•13 years ago
|
||
Comment on attachment 460181 [details] [diff] [review] Patch (In reply to comment #0) > This appears to have fallen out of disuse sometime before we left CVS. I > didn't try too hard to track its origins down but the macros and the interface > have been unused for a long time. That's rather irrelevant because this is an API for extensions. I don't know of any users anymore, though I guess it'd be nice to at least notify about this somewhere (newsgroup post?). >- { &kNS_BASE_DOM_EXCEPTION_CID, false, NULL, nsBaseDOMExceptionConstructor }, You need to remove nsBaseDOMExceptionConstructor too (NS_GENERIC_FACTORY_CONSTRUCTOR(nsBaseDOMException)).
Attachment #460181 -
Flags: review?(peterv) → review+
Reporter | ||
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → INCOMPLETE
Assignee | ||
Comment 3•12 years ago
|
||
I want to kill this too
Assignee: khuey → Ms2ger
Attachment #460181 -
Attachment is obsolete: true
Status: RESOLVED → REOPENED
Attachment #614170 -
Flags: review?(peterv)
Resolution: INCOMPLETE → ---
Comment 4•12 years ago
|
||
(In reply to Ms2ger from comment #3) > I want to kill this too So you did notify about this somewhere? I'd rather make sure nobody uses this stuff.
Comment 5•12 years ago
|
||
Comment on attachment 614170 [details] [diff] [review] Patch Waiting for reply on comment 4.
Attachment #614170 -
Flags: review?(peterv) → review-
Assignee | ||
Comment 6•11 years ago
|
||
Looks like someone beat me to removing it.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 11 years ago
Resolution: --- → WORKSFORME
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•