Closed
Bug 397813
Opened 17 years ago
Closed 17 years ago
remove nsFontList/nsIFontList
Categories
(Core Graveyard :: GFX, defect)
Core Graveyard
GFX
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dbaron, Assigned: dbaron)
Details
Attachments
(1 file)
|
(deleted),
patch
|
pavlov
:
review+
pavlov
:
superreview+
pavlov
:
approval1.9+
|
Details | Diff | Splinter Review |
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?
|
Assignee | |
Comment 1•17 years ago
|
||
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)
|
||
Updated•17 years ago
|
Attachment #282607 -
Flags: superreview+
Attachment #282607 -
Flags: review?(pavlov)
Attachment #282607 -
Flags: review+
Attachment #282607 -
Flags: approval1.9+
|
|
Assignee | |
Comment 2•17 years ago
|
||
Checked in to trunk.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
||
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•