Outparamdel nsIParser::GetStreamListener

RESOLVED FIXED in mozilla11

Status

()

Core
HTML: Parser
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla11
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
There's no point in using an out-param (and shaving off a line or two from nsHTMLDocument::StartDocumentLoad is always nice). I'll fix if nobody beats me to it.
(Assignee)

Comment 1

6 years ago
Note to self: mentioned on <https://developer.mozilla.org/en/Gecko/HTML_parser_threading>.
(Assignee)

Comment 2

6 years ago
Created attachment 570632 [details] [diff] [review]
Patch v1
Assignee: nobody → Ms2ger
Status: NEW → ASSIGNED
Attachment #570632 - Flags: review?(hsivonen)
Comment on attachment 570632 [details] [diff] [review]
Patch v1

r=hsivonen if you revise the nsIParser interface ID.
Attachment #570632 - Flags: review?(hsivonen) → review+
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/6ae59761dc37
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Whiteboard: [qa-]
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.