1714561 - Allow single-zone JS holders to hold pointers into the atoms zone

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Jon Coppeard (:jonco)]

In the browser we try to partition JS holders by the zone of their JS referents, so that we can speed up marking by skipping pointers into zones that are not being collected.

Strings complicate this because any string might be in the atoms zone. We should allow pointers into the atoms zone from otherwise single-zone holders and make sure we mark all holders when collecting the atoms zone.

Comment 1

[Jon Coppeard (:jonco)]

Attachment: Bug 1714561 - Allow single-zone JS holders to contain pointers into the atoms zone r=mccr8!,sfink!

Comment 3

[Jon Coppeard (:jonco)]

Attachment: Bug 1714561 - Add test that single-zone JS holders can contain a pointer into the atoms zone r?mccr8

Depends on D116848

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3061d812f68a
Allow single-zone JS holders to contain pointers into the atoms zone r=mccr8,sfink
https://hg.mozilla.org/integration/autoland/rev/2e7f08395e60
Add test that single-zone JS holders can contain a pointer into the atoms zone r=mccr8

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/20750faf42a3
Fix black lint. r=fix CLOSED TREE

Comment 6

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/3061d812f68a
https://hg.mozilla.org/mozilla-central/rev/2e7f08395e60
https://hg.mozilla.org/mozilla-central/rev/20750faf42a3