Closed Bug 948466 Opened 6 years ago Closed 5 years ago

Rename gfxPangoFonts to gfxFontconfigFonts

Categories

(Core :: Graphics, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed

People

(Reporter: jrmuizel, Assigned: belleville.nicolas)

Details

Attachments

(1 file, 1 obsolete file)

This more accurately reflects what it does and how it works.
Hi Jeff,
I would like to work on this bug
Great.
Assignee: nobody → belleville.nicolas
Attached patch patch that solve bug 948466 (obsolete) — Splinter Review
Here is a patch for this bug
Attachment #8608134 - Flags: review?(nical.bugzilla)
Comment on attachment 8608134 [details] [diff] [review]
patch that solve bug 948466

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

Please use hg mv to rename files in order to preserve the mercurial history. Other than that, looks good.
Attachment #8608134 - Flags: review?(nical.bugzilla) → review-
New patch for this bug
Attachment #8608134 - Attachment is obsolete: true
Attachment #8608538 - Flags: review?(nical.bugzilla)
Comment on attachment 8608538 [details] [diff] [review]
patch that solve bug 948466

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

Looks good, i'll push this to try before landing.
Attachment #8608538 - Flags: review?(nical.bugzilla) → review+
https://hg.mozilla.org/mozilla-central/rev/36d1a8d3936a
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Just a note here, the code that's been shifted here will eventually be phased out when the new fontconfig fontlist code is used by default for release builds. See bug 1119128.
You need to log in before you can comment on or make changes to this bug.