Open Bug 563890 Opened 10 years ago Updated Last year

Remove nsHTMLContentSink from the tree

Categories

(Core :: DOM: HTML Parser, enhancement, P4)

enhancement

Tracking

()

People

(Reporter: hsivonen, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

nsHTMLContentSink should be removed as dead code once it is no longer used.
Priority: -- → P4
Depends on: 636692
Depends on: sync-about-blank
Blocks: 650775
Depends on: 596182
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #530609 - Flags: review?(jonas)
Blocks: 655255
I like!

There's a test for the interface isindex elements implement with the old parser; it looks like you missed it.
Depends on: 655701
(In reply to comment #2)
> There's a test for the interface isindex elements implement with the old
> parser; it looks like you missed it.

There's a whole bunch of tests that need editing. Filed bug 655701 about those.
Attachment #530609 - Attachment is obsolete: true
Attachment #560571 - Flags: review?(mrbkap)
Attachment #530609 - Flags: review?(jonas)
Comment on attachment 560571 [details] [diff] [review]
Remove the old sink and prefs for taking code paths that used it, rebased

Review of attachment 560571 [details] [diff] [review]:
-----------------------------------------------------------------

Woo!
Attachment #560571 - Flags: review?(mrbkap) → review+
Depends on: 650787
No longer depends on: 650787
No longer blocks: 655255
It's been six months since review - was this forgotten or it's waiting for something I can't see?
(In reply to Radek 'sysKin' Czyz from comment #6)
> It's been six months since review - was this forgotten or it's waiting for
> something I can't see?

It's waiting for its dependencies. At this point, bug 543435 is the one left.
Assignee: hsivonen → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.