Merge SweepRegExps with SweepMisc

RESOLVED FIXED in Firefox 61

Status

()

enhancement
P3
normal
RESOLVED FIXED
a year ago
10 months ago

People

(Reporter: anba, Assigned: sumi29, Mentored)

Tracking

({good-first-bug})

Trunk
mozilla61
Points:
---

Firefox Tracking Flags

(firefox59 wontfix, firefox60 wontfix, firefox61 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

a year ago
https://mozilla.logbot.info/jsapi/20180118#c14152420

SweepRegExps is no longer performing any complicated computations (bug 1345177), so we should merge it with SweepMisc to avoid any additional AutoRunParallelTask  overhead.
Keywords: good-first-bug
Priority: -- → P3
Mentor: jcoppeard
Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8971733 [details]
Bug 1431434 - Merge SweepRegExps with SweepMisc;

https://reviewboard.mozilla.org/r/240496/#review246262

Looks good.  Thanks for this fix!
Attachment #8971733 - Flags: review?(jcoppeard) → review+
Sumit, I assume from comments on other bugs that you don't have commit access, so I'll land this.
Assignee: nobody → sumi29

Comment 4

a year ago
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a2409475579a
Merge SweepRegExps with SweepMisc; r=jonco

Comment 5

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a2409475579a
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.