Closed Bug 289316 Opened 20 years ago Closed 20 years ago

[FIXr]Remove unused aDeepSetDocument args on nsIContent

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla1.8beta2

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(1 file, 1 obsolete file)

nsIContent::AppendChildTo and nsIContent::InsertChildAt now have unused aDeepSetDocument args. Those should be removed.
Assignee: general → bzbarsky
OS: Linux → All
Priority: -- → P1
Hardware: PC → All
Target Milestone: --- → mozilla1.8beta2
Depends on: 286000
Attached patch Remove the suckers (obsolete) — Splinter Review
Attachment #180011 - Flags: superreview?(jst)
Attachment #180011 - Flags: review?(bugmail)
Summary: Remove unused aDeepSetDocument args on nsIContent → [FIX]Remove unused aDeepSetDocument args on nsIContent
Attachment #180011 - Flags: superreview?(jst) → superreview+
Comment on attachment 180011 [details] [diff] [review] Remove the suckers Requesting approval... Very straightforward patch, just removes unused code.
Attachment #180011 - Flags: approval1.8b2?
Summary: [FIX]Remove unused aDeepSetDocument args on nsIContent → [FIXr]Remove unused aDeepSetDocument args on nsIContent
Comment on attachment 180011 [details] [diff] [review] Remove the suckers a=brendan for 1.8b2. /be
Attachment #180011 - Flags: approval1.8b2? → approval1.8b2+
Attached patch Updated to tipSplinter Review
Attachment #180011 - Attachment is obsolete: true
Fixed.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: