1874241 - Make recordings cache current draw target rather than include for each event

Status: RESOLVED FIXED

(Core :: Graphics, enhancement, P3)

Description

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

Most of the time when we record an event, it is for the same DrawTarget that we just used. We could save a good chunk of recording size by adding a new event RecordedSetCurrentDrawTarget to set the current DrawTarget whenever it changes and make all of the other events use the current draw target.

Comment 1

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

Attachment: Bug 1874241 - Cache current DrawTarget in recordings.

Since most draw commands in a recording reference the same DrawTarget
over and over again, it saves a lot of space in the recording buffer to
create a dedicated RecordedSetCurrentDrawTarget command which caches the
current DrawTarget pointer and each subsequent command that needs a
DrawTarget will use the current.

Depends on D198295

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8872820f3964
Cache current DrawTarget in recordings. r=gfx-reviewers,lsalzman

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/8872820f3964