Closed Bug 397813 Opened 17 years ago Closed 17 years ago

remove nsFontList/nsIFontList

Categories

(Core Graveyard :: GFX, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dbaron, Assigned: dbaron)

Details

Attachments

(1 file)

nsFontList/nsIFontList is a wrapper for the platform-specific nsIFontEnumerator implementations that has an extra two layers of COM.  It's not used anywhere in our tree, and it's not in the factory for thebes gfx, so it's no longer (with thebes) possible for anyone outside to create one.

We should either list it in the thebes gfx factory or remove it.  I propose removing it.

Does that sound reasonable?
Attached patch patch to removeSplinter Review
I wonder whether this was used by embedders at some point, given the tests in embedding/qa/.  But the interface was never marked frozen or under review, and it is just a wrapper for nsIFontEnumerator...
Attachment #282607 - Flags: review?(pavlov)
Attachment #282607 - Flags: superreview+
Attachment #282607 - Flags: review?(pavlov)
Attachment #282607 - Flags: review+
Attachment #282607 - Flags: approval1.9+
Checked in to trunk.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.