Closed
Bug 445685
Opened 16 years ago
Closed 16 years ago
Improve performance with large numbers of text fragments
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: longsonr, Assigned: longsonr)
References
Details
(Keywords: perf)
Attachments
(1 file)
3.04 KB,
patch
|
jwatt
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
Currently we do for each glyph fragment within a text calculate the initial position of all glyph fragments Avoiding the for loop and just calculating the initial position once per text speeds things up. Also wait for the InitialUpdate before allowing DidSetStyleContext updates as that is O(N<sup>2</sup>) too.
Attachment #329963 -
Flags: review?(jwatt)
Comment 1•16 years ago
|
||
Comment on attachment 329963 [details] [diff] [review] patch Nice. r=jwatt
Attachment #329963 -
Flags: review?(jwatt) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #329963 -
Flags: superreview?(mats.palmgren)
Assignee | ||
Updated•16 years ago
|
Attachment #329963 -
Flags: superreview?(mats.palmgren) → superreview?(roc)
Attachment #329963 -
Flags: superreview?(roc) → superreview+
Assignee | ||
Comment 2•16 years ago
|
||
checked in 2e445c864722.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•