The default bug view has changed. See this FAQ.

Remove old speculative HTML parsing code

RESOLVED FIXED in mozilla12

Status

()

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

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

Trunk
mozilla12
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Get rid of the old speculative HTML parsing code. It's not used for XML anyway and it doesn't make sense to run it for about:blank.
(Assignee)

Comment 1

5 years ago
Created attachment 587651 [details] [diff] [review]
Remove obsolete speculation code
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
(Assignee)

Updated

5 years ago
Attachment #587651 - Flags: review?(mrbkap)
Comment on attachment 587651 [details] [diff] [review]
Remove obsolete speculation code

It makes me sad to review this, but it's time to move on. That being said, the Holder class (just above nsSpeculativeScriptThread) can be deleted as well.
Attachment #587651 - Flags: review?(mrbkap) → review+
(Assignee)

Comment 3

5 years ago
Thanks. Landed with Holder removed, too.
https://hg.mozilla.org/integration/mozilla-inbound/rev/39e49fcaed22
https://hg.mozilla.org/mozilla-central/rev/39e49fcaed22
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.