Closed
Bug 1743849
Opened 3 years ago
Closed 3 years ago
Separate pinning from atomization
Categories
(Core :: JavaScript: GC, task, P3)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
96 Branch
| Tracking | Status | |
|---|---|---|
| firefox96 | --- | fixed |
People
(Reporter: jonco, Assigned: jonco)
Details
Attachments
(2 files)
Following bug 1742591 we can separate out the APIs for pinning atoms (which are not widely used) from atomization. This simplifies the atomization code.
|
Assignee | |
Comment 1•3 years ago
|
||
I don't know why these are pinned and I don't see any reason for it.
|
|
Assignee | |
Comment 2•3 years ago
|
||
This removes the pinning parameter from atomization calls, which do not pin.
Atoms can be pinned afterwards with PinAtom. Public APIs are unchanged.
Depends on D132617
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/15cb1810bd76
Part 1: Remove pinning from some debugger atoms r=jandem
https://hg.mozilla.org/integration/autoland/rev/9c1f1875f6c6
Part 2: Separate internal APIs for atomization and pinning r=jandem
|
||
Comment 4•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/15cb1810bd76
https://hg.mozilla.org/mozilla-central/rev/9c1f1875f6c6
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox96:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•