Closed
Bug 500556
Opened 16 years ago
Closed 16 years ago
Modifying textnodes causes unnecessary reframing
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: roc, Assigned: roc)
References
Details
Attachments
(2 files)
|
540 bytes,
text/html
|
Details | |
|
10.28 KB,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
The attached testcase is slower than necessary on trunk. It changes the contents of the text node in a block 100,000 times. It's 3x slower for white-space:normal than white-space:pre in my debug build. In the white-space:normal case we keep recreating the frame for the text because we've marked the text node with FRAMETREE_DEPENDS_ON_CHARS. We can fix this using the scheme in bug 495385 comment #57.
|
Assignee | |
Comment 1•16 years ago
|
||
Attachment #385291 -
Flags: superreview?(bzbarsky)
Attachment #385291 -
Flags: review?(bzbarsky)
|
||
Comment 2•16 years ago
|
||
Comment on attachment 385291 [details] [diff] [review]
fix
Makes sense.
Attachment #385291 -
Flags: superreview?(bzbarsky)
Attachment #385291 -
Flags: superreview+
Attachment #385291 -
Flags: review?(bzbarsky)
Attachment #385291 -
Flags: review+
|
|
Assignee | |
Comment 3•16 years ago
|
||
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
•