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)

defect

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.)
Status: NEW → ASSIGNED
Target Milestone: M15
Keywords: perf
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...
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
Keywords: mozilla0.8
nom, 0.8.
Keywords: mozilla0.8.1
Keywords: mozilla0.8
Keywords: mozilla0.9.1
erik resign. reassign all his bug to ftang for now.
Assignee: erik → ftang
Status: ASSIGNED → NEW
mark all future new as assigned after move from erik to ftang
Status: NEW → ASSIGNED
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.