Open Bug 934565 Opened 11 years ago Updated 3 years ago

Don't clear op queue all the time

Categories

(Core :: DOM: HTML Parser, defect, P5)

x86_64
Linux
defect

Tracking

()

People

(Reporter: smaug, Unassigned)

References

Details

Attachments

(1 file)

Attached patch WIPSplinter Review
Currently we pretty effectively clear mOpQueues once we have finished parsing.
Then when we next time parse again we need to do bunch of allocations.

The patch caches too much and I didn't even look at in which cases Clear() would be fine. Just replaced all the explicit .Clear() calls with ClearOps().

The test results for bug 922018 varies a lot, but ~5% speed up here.

Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.

Severity: normal → S4
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: