Closed Bug 1736604 Opened 1 month ago Closed 21 days ago

Remove the GCPolicy and container sweep methods

Categories

(Core :: JavaScript: GC, task, P3)

task

Tracking

()

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: jonco, Assigned: jonco, NeedInfo)

References

Details

Attachments

(3 files)

There's one last use of these, and then we can remove the duplicated methods entirely. From now on everything can use traceWeak instead of needsSweep/sweep.

This functionality has been replaced by the traceWeak methods.

Depends on D128901

It's only used for traceWeak it's true, but it's like a GCPolicy for the map
entry itself.

Depends on D128902

Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e1ef1c085097
Part 1: Replace sweeping with tracing weak edges in WeakRef implementation r=sfink
https://hg.mozilla.org/integration/autoland/rev/31b0cbcc6e84
Part 2: Remove GCPolicy and container needsSweep and sweep methods r=sfink
https://hg.mozilla.org/integration/autoland/rev/3ef8aa197b0a
Part 3: Rename MapSweepPolicy to MapEntryGCPolicy r=sfink
Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
Status: RESOLVED → REOPENED
Flags: needinfo?(jcoppeard)
Resolution: FIXED → ---
Target Milestone: 95 Branch → ---
Attachment #9246669 - Attachment description: Bug 1736604 - Part 1: Replace sweeping with tracing weak edges in WeakRef implementation r?sfink → Bug 1736604 - Part 1: Replace sweeping with tracing weak edges in WeakRef implementation r=sfink
Attachment #9246670 - Attachment description: Bug 1736604 - Part 2: Remove GCPolicy and container needsSweep and sweep methods r?sfink → Bug 1736604 - Part 2: Remove GCPolicy and container needsSweep and sweep methods r=sfink
Attachment #9246672 - Attachment description: Bug 1736604 - Part 3: Rename MapSweepPolicy to MapEntryGCPolicy r?sfink → Bug 1736604 - Part 3: Rename MapSweepPolicy to MapEntryGCPolicy r=sfink
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/10d25724c8be
Part 1: Replace sweeping with tracing weak edges in WeakRef implementation r=sfink
https://hg.mozilla.org/integration/autoland/rev/820758a84c4d
Part 2: Remove GCPolicy and container needsSweep and sweep methods r=sfink
https://hg.mozilla.org/integration/autoland/rev/3f566bda25e0
Part 3: Rename MapSweepPolicy to MapEntryGCPolicy r=sfink
Status: REOPENED → RESOLVED
Closed: 1 month ago21 days ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
You need to log in before you can comment on or make changes to this bug.