Closed Bug 1507687 Opened 2 years ago Closed 2 years ago

add static atoms for all identifiers in user agent style sheets

Categories

(Core :: XPCOM, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: heycam, Assigned: heycam)

References

Details

Attachments

(1 file)

There are a number of dynamic atoms that always get created, since they're found in user agent style sheets.  We should add static atoms for these.  (This is necessary for the bug 1474793 work too, where we can only store static atoms in shared memory.)
Assignee: nobody → cam
Status: NEW → ASSIGNED
Pushed by cmccormack@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9135ee87f8d9
Add static atoms for strings in UA style sheets r=njn
https://hg.mozilla.org/mozilla-central/rev/9135ee87f8d9
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Do we have a document or static asserts to remind people to add static atoms if they introduce new strings in UA style sheets?
There are no assertions currently, but if/when the shared UA style sheets work land, then there will be assertions as part of that.
You need to log in before you can comment on or make changes to this bug.