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)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
mozilla80
Tracking | Status | |
---|---|---|
firefox80 | --- | fixed |
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
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 | ||
Comment 1•4 years ago
|
||
Updated•4 years ago
|
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
Comment 3•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox80:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in
before you can comment on or make changes to this bug.
Description
•