Closed Bug 1176064 Opened 5 years ago Closed 5 years ago

Do processing of the jstest's skiplist inline

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

Right now we are splitting the tests list into a skipped and a non-skipped test list. Even if we turn everything into a generator, if we iterate the skip list to append results we'll still have to serialize. I think the best way forward is to keep everything in a single list and do the skip processing inside the task runners.
Attachment #8624463 - Flags: review?(sphink)
Comment on attachment 8624463 [details] [diff] [review]
inline_skiplist_processing-v0.diff

Review of attachment 8624463 [details] [diff] [review]:
-----------------------------------------------------------------

nice
Attachment #8624463 - Flags: review?(sphink) → review+
https://hg.mozilla.org/mozilla-central/rev/23684898a457
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.