Closed
Bug 26670
Opened 25 years ago
Closed 23 years ago
need to remove the slow pref enumeration in font engine
Categories
(Core :: Layout, defect, P3)
Core
Layout
Tracking
()
RESOLVED
WONTFIX
Future
People
(Reporter: erik, Assigned: ftang)
Details
(Keywords: perf)
Troy pointed out that the font pref enumeration that we are currently doing
when the font for the document's language group does not contain a glyph for the
current character is really slow. We should remove the call to that enumerator
and do it in a better way, possibly by hard-coding the language groups, since
we're already hard-coding them in the font enumerator (for font prefs UI)
anyway. Or try to find an extensible way to do that. Also, instead of some
random order for those languages, it might be a good idea to try the user's
language first. (Remember, this is *after* the document's language has failed.)
Reporter | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M15
Reporter | ||
Comment 1•25 years ago
|
||
Prioritizing my bugs. It is not worth changing this until it actually shows up
on the performance radar, i.e. in Quantify's output. Therefore M20.
Target Milestone: M15 → M20
No No No. That was not part of the deal. You were supposed to fix the problem,
not try to ignore it and wait for me to prove its a performance problem.
Even if that code only executes once, it's inefficient. And if it's never
executed, then it's dead code and should be removed. BTW, that's my solution to
how to fix the problem...
Assignee | ||
Comment 3•24 years ago
|
||
Do we still need to do this? Move it to future for now. If we need to deal with
it, we should split it into two bugs- one for window and one for linux.
Target Milestone: M20 → Future
Updated•24 years ago
|
Keywords: mozilla0.8
Comment 4•24 years ago
|
||
nom, 0.8.
Updated•24 years ago
|
Keywords: mozilla0.8.1
Updated•24 years ago
|
Keywords: mozilla0.8
Updated•24 years ago
|
Keywords: mozilla0.9.1
Assignee | ||
Comment 5•24 years ago
|
||
erik resign. reassign all his bug to ftang for now.
Assignee: erik → ftang
Status: ASSIGNED → NEW
Assignee | ||
Comment 6•24 years ago
|
||
mark all future new as assigned after move from erik to ftang
Status: NEW → ASSIGNED
Assignee | ||
Comment 7•23 years ago
|
||
no body know what this bug really mean now. mark it as wontfix
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•