Remove dead code from HTMLContentSink

RESOLVED FIXED in mozilla12

Status

()

RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla12
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(9 attachments)

(Assignee)

Description

7 years ago
Created attachment 590467 [details] [diff] [review]
Part a: frameset - link - style - doctype
Attachment #590467 - Flags: review?(hsivonen)
(Assignee)

Updated

7 years ago
Attachment #590467 - Attachment description: Part a: fameset - link - style - doctype → Part a: frameset - link - style - doctype
(Assignee)

Comment 1

7 years ago
Created attachment 590470 [details] [diff] [review]
Part b: AddDocTypeDecl
Attachment #590470 - Flags: review?(hsivonen)
(Assignee)

Comment 2

7 years ago
Created attachment 590472 [details] [diff] [review]
Part c: mNumOpenIFRAMES

I can't even find a tree on MXR where this was used for anything
Attachment #590472 - Flags: review?(hsivonen)
(Assignee)

Comment 3

7 years ago
Created attachment 590481 [details] [diff] [review]
Part d: mInsideNoXXXTag
Attachment #590481 - Flags: review?(hsivonen)
(Assignee)

Comment 4

7 years ago
Created attachment 590482 [details] [diff] [review]
Part e: meta
Attachment #590482 - Flags: review?(hsivonen)
(Assignee)

Comment 5

7 years ago
Created attachment 590483 [details] [diff] [review]
Part f: form
Attachment #590483 - Flags: review?(hsivonen)
(Assignee)

Comment 6

7 years ago
Created attachment 590484 [details] [diff] [review]
Part g: IsFormOnStack
Attachment #590484 - Flags: review?(hsivonen)
(Assignee)

Comment 7

7 years ago
Created attachment 590485 [details] [diff] [review]
Part h: AddProcessingInstruction
Attachment #590485 - Flags: review?(hsivonen)
(Assignee)

Comment 8

7 years ago
Created attachment 590486 [details] [diff] [review]
Part i: AddComment
Attachment #590486 - Flags: review?(hsivonen)
Attachment #590467 - Flags: review?(hsivonen) → review+
Comment on attachment 590470 [details] [diff] [review]
Part b: AddDocTypeDecl

In theory, you should revise the IID of nsIHTMLContentSink.
Attachment #590470 - Flags: review?(hsivonen) → review+
Attachment #590472 - Flags: review?(hsivonen) → review+
Attachment #590481 - Flags: review?(hsivonen) → review+
Attachment #590482 - Flags: review?(hsivonen) → review+
Attachment #590483 - Flags: review?(hsivonen) → review+
Comment on attachment 590484 [details] [diff] [review]
Part g: IsFormOnStack

Theoretical IID bump.
Attachment #590484 - Flags: review?(hsivonen) → review+
Attachment #590485 - Flags: review?(hsivonen) → review+
Comment on attachment 590486 [details] [diff] [review]
Part i: AddComment

Thanks for removing this cruft!

BTW, nsLoggingSink is only useful for diagnostics while fixing this bug. If you feel you don't need nsLoggingSink diagnostics for fixing this bug, feel free to delete nsLoggingSink altogether.
Attachment #590486 - Flags: review?(hsivonen) → review+
You need to log in before you can comment on or make changes to this bug.