Closed Bug 1007013 Opened 6 years ago Closed 6 years ago

eliminate gfxFT2FontGroup

Categories

(Core :: Graphics: Text, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: jtd, Assigned: jtd)

Details

Attachments

(2 files)

This is a follow-up from bug 684889 which refactored the Android font code. At this point in time we don't build gfxFT2Group code on any platform, nor is it referenced anywhere in the source tree.  So I think we should just trim this out so it doesn't show up when doing searches within gfx code.
Remove unused gfxFT2FontGroup code which is no longer referenced on by any platform.
Attachment #8419134 - Flags: review?(m_kato)
Comment on attachment 8419134 [details] [diff] [review]
remove-gfxFT2FontGroup.patch

This code seems to be no longer used after bug 947379.
Attachment #8419134 - Flags: review?(m_kato) → review+
Shouldn't you remove it from gfxFT2Fonts.h as well, then?
Yup! Remove from header.
Attachment #8419213 - Flags: review?(m_kato)
Attachment #8419213 - Flags: review?(m_kato) → review+
https://hg.mozilla.org/mozilla-central/rev/d002d113feca
https://hg.mozilla.org/mozilla-central/rev/996575e5352b
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.