Closed Bug 1147083 Opened 10 years ago Closed 10 years ago

We're recreating empty PaintedLayers on every frame for layers that have been optimized to ColorLayers / ImageLayers

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: mstange, Assigned: mstange)

Details

(Keywords: regression)

This is a regression from bug 913443. (In reply to Markus Stange [:mstange] from bug 913443 comment #2) > This setup can create new layers even when there would have been an unused > layer available for recycling. But that case can only occur when the old and > the new layer share no common display items, so everything would be > invalidated anyway. This is the case we're hitting: the old layer and the new layer have no display items in common, because they're both empty. We might not even need a PaintedLayer in this case in the first place.
I was wrong. The PaintedLayer that I was seeing getting recreated all the time wasn't one that was optimized away. It was one that was created for an event region display item, and those don't even hit the "recycle old layer" code path. I'll file a new bug about that, so as not to confuse matters. Recycling of optimized away layers works.
No longer blocks: 913443
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.