1818488 - Hoist access to gfxContext::GetTextDrawer() out of the glyph-rendering inner loop

Status: RESOLVED FIXED

(Core :: Graphics: Text, defect)

Description

[Jonathan Kew [:jfkthame]]

Noticed in profiles such as https://share.firefox.dev/3K8L3te that calling GetTextDrawer() is a (small but) measurable cost. We can hoist this up into the TextRunDrawParams and only retrieve it once, instead of per-glyph.

Comment 1

[Jonathan Kew [:jfkthame]]

Attachment: Bug 1818488 - Hoist access to gfxContext::GetTextDrawer() out of the glyph-rendering inner loop. r=#gfx-reviewers

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c76726c62951
Hoist access to gfxContext::GetTextDrawer() out of the glyph-rendering inner loop. r=gfx-reviewers,lsalzman

Comment 3

[Cosmin Sabou [:CosminS]]

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