The default bug view has changed. See this FAQ.

stylo: Style sharing cache not working

Assigned to



CSS Parsing and Computation
2 months ago
13 days ago


(Reporter: bholley, Assigned: emilio)


(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)



(1 attachment)

One discovery in bug 1331848 and the logging in bug 1331856 is that the style sharing doesn't appear to be getting populated at all.

There are potentially several reasons for this. Emilio pointed out that the cache is optimized for parallel traversal, and probably doesn't do the right thing for sequential traversal. My logging also indicates that we never insert any elements into the cache because of "animations", which is probably wrong.
NI emilio to investigate!
Flags: needinfo?(emilio+bugs)
Created attachment 8828614 [details] [diff] [review]
Improve logging a tiny bit.

Please fold this into your patch if you can, since we might as well get it

Comment 3

2 months ago
I'm trying to land a preliminar patch for this at

Comment 4

2 months ago
(Will probably land the logging patch as part of the rest of the style sharing cache changes)
This is probably p1 because it has a large impact on the performance measurements we'll get out of the style system. Let me know if it looks like it will take a lot of time and we can re-evaluate.
Priority: -- → P1
Assignee: nobody → emilio+bugs

Comment 6

13 days ago
Some initial cleanup at

I plan to make some improvements to the current cache before implementing the sequential logic.
Flags: needinfo?(emilio+bugs)
You need to log in before you can comment on or make changes to this bug.