1724727 - SVG blob image recordings should take visibility into account

Status: RESOLVED FIXED

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

Description

[Andrew Osmond [:aosmond] (he/him)]

Right now, SVG image recordings just use a clip rect to trim out the parts of the recording they don't need. We should switch this to the visibility rect for the display list builder so that it doesn't even try to record display items outside the rect. This will make recordings significantly smaller and faster to paint.

Comment 1

[Andrew Osmond [:aosmond] (he/him)]

Attachment: Bug 1724727 - Make SVG image blob recordings use visibility rects.

Supplying a visibility rect to the display list builder when recording
an SVG image blob will allow the display list builder to trim out
unnecessary items and make the recordings smaller. This is particularly
important when we have an SVG image used as an atlas.

This patch also simplifies the call path for the recordings. This allows
us to avoid unnecessary clips and transforms and shrink the recording
further, as well as make the code more auditable.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fdeba1ebcb4d
Make SVG image blob recordings use visibility rects. r=jrmuizel

Comment 3

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/fdeba1ebcb4d