Closed Bug 1649527 Opened 4 years ago Closed 4 years ago

Avoid unnecessary cmap-loading when the shared font-list already has the character map

Categories

(Core :: Layout: Text and Fonts, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla80
Tracking Status
firefox80 --- fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file)

We store character-map records for font character coverage in the shared font list Face entries, to avoid needing separate per-process copies of them. But I notice that when we instantiate a new gfxFontEntry for a face, the code fails to check if there is already a character map stored for it, and so we may redundantly re-read the cmap table. We shouldn't do that.

Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7db7ff639884
Initialize character map when creating a gfxFontEntry if it is already available from a shared font-list Face record. r=jwatt
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: