Closed Bug 1818149 Opened 2 years ago Closed 2 years ago

Hoist call to textDrawer->GeckoClipRect() out of the glyph-painting inner loop

Categories

(Core :: Graphics: Text, defect)

defect

Tracking

()

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file)

This is just a minor followup optimization for bug 1817184. I noticed that TextDrawTarget::GeckoClipRect() showed up (slightly) in profiles while working on bug 1816927. So rather than calling this for every glyph, we should do it once at the textrun level and pass the rect down in the drawParams.

Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/71a1eb34bc90 Hoist access to textDrawer->GeckoClipRect() out of the inner DrawOneGlyph method. r=gfx-reviewers,lsalzman
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: