Closed Bug 1355791 Opened 7 years ago Closed 7 years ago

Stop pushing so many scroll layers

Categories

(Core :: Graphics: WebRender, enhancement, P3)

Other Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

Mostly the reason we're pushing scroll layers right now is so that we can tell webrender about clip masks. But we can do that using the clip on the display item. So we should do that instead, and stop pushing scroll layers. We can do this for most of the WebRender*Layer types, the only exception is the WebRenderContainerLayer which I'll deal with later.

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=bf943bff326bbd91b81f8789fadecec2549de8f4
Comment on attachment 8857458 [details]
Bug 1355791 - Instead of pushing a scroll layer, put the clip mask into the existing item clip.

https://reviewboard.mozilla.org/r/129444/#review132002
Attachment #8857458 - Flags: review+
Attachment #8857458 - Flags: review?(mchang)
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/projects/graphics/rev/fc48bf789905
Instead of pushing a scroll layer, put the clip mask into the existing item clip. r=jrmuizel
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: