If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Sweep all WeakCaches in parallel

RESOLVED FIXED in Firefox 48

Status

()

Core
JavaScript: GC
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

Trunk
mozilla48
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Created attachment 8740185 [details] [diff] [review]
sweep_weakcaches_in_parallel-v0.diff

Given that we are already sweeping everything in parallel, we can't move stuff over to the new system without regressing things until we have equivalent parallelism there too. This wasn't actually that bad, surprisingly.
Attachment #8740185 - Flags: review?(sphink)
(Assignee)

Updated

2 years ago
Blocks: 1263771
(Assignee)

Updated

2 years ago
Blocks: 1263772
(Assignee)

Updated

2 years ago
Blocks: 1263777
Comment on attachment 8740185 [details] [diff] [review]
sweep_weakcaches_in_parallel-v0.diff

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

This is nice.
Attachment #8740185 - Flags: review?(sphink) → review+
(Assignee)

Comment 2

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b23a6286c125783b582eb59967ab5e574133478a
Bug 1263769 - Sweep WeakCaches in parallel; r=sfink

Comment 3

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b23a6286c125
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox48: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
(Assignee)

Updated

2 years ago
Blocks: 1265483
(Assignee)

Updated

a year ago
Blocks: 1266552
Depends on: 1280132
You need to log in before you can comment on or make changes to this bug.