Closed Bug 1479010 Opened Last year Closed Last year

Avoid spending time in PopulateCairoScaledFont()

Categories

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

63 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: jrmuizel, Assigned: lsalzman)

References

Details

Attachments

(1 file)

Flags: needinfo?(lsalzman)
When creating NFRs, the only time we should actually need to populate the cairo scaled font is when we are printing to DTCairo, otherwise there is no need.

This also fixes the same issue on Mac.
Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
Flags: needinfo?(lsalzman)
Attachment #8996070 - Flags: review?(jmuizelaar)
Priority: -- → P3
Comment on attachment 8996070 [details] [diff] [review]
don't populate a ScaledFont's cairo scaled font unless targeting cairo

Review of attachment 8996070 [details] [diff] [review]:
-----------------------------------------------------------------

I thought I had already r+ this.
Attachment #8996070 - Flags: review?(jmuizelaar) → review+
Blocks: 1480196
Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3686ee636ac2
don't populate a ScaledFont's cairo scaled font unless targeting cairo. r=jrmuizel
https://hg.mozilla.org/mozilla-central/rev/3686ee636ac2
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.