[Webkit-unassigned] [Bug 37292] http://trac.webkit.org/changeset/57215 caused perf/memory regressions

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=37292





--- Comment #11 from mitz at webkit.org  2010-04-12 10:55:22 PST ---
(In reply to comment #9)
> Yeah I really should not have put r+ on the original bug.  I'll take some of
> the blame for that.  It was obvious that the bug was going to regress the
> performance of the complex text code path, and I shouldn't have let that slide.

According to Ojan’s testing, that is not correct. The complex path has not
become measurably slower.

> We should probably do a surgical patch that disables this tracking without
> removing all the code from the tree, so that we can get the performance back.

I think the thinks to consider now are:
1) Add a third code path (fast w/glyph overflow) for the “stacked diacritics”
range
or if that turns out to be too hard
2) #ifdef-out the change to canUseGlyphCache()

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.