Closed
Bug 1263769
Opened 9 years ago
Closed 9 years ago
Sweep all WeakCaches in parallel
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla48
| Tracking | Status | |
|---|---|---|
| firefox48 | --- | fixed |
People
(Reporter: terrence, Assigned: terrence)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
sfink
:
review+
|
Details | Diff | Splinter Review |
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)
|
||
Comment 1•9 years ago
|
||
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•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b23a6286c125783b582eb59967ab5e574133478a
Bug 1263769 - Sweep WeakCaches in parallel; r=sfink
|
||
Comment 3•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•