Open
Bug 86950
Opened 24 years ago
Updated 1 month ago
make floats not require state recovery
Categories
(Core :: Layout: Floats, task, P3)
Core
Layout: Floats
Tracking
()
NEW
Future
People
(Reporter: dbaron, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: perf, Whiteboard: [whitebox])
Right now we create and destroy space managers during each reflow (for every
frame that could possibly need a space manager). This leads to O(N^2) state
recovery for an incremental reflow. This seems complicated to fix, but some
thoughts on how it could be done are:
* store the frame that would have the space manager in the reflow state rather
than (or in addition to) the space manager itself, and store the space manager
for the lifetime of the frame tree as a property of the frame, but only for
frames that actually contain floats.
* (the hard part, perhaps) Keep the space manager updated when things change
between reflows
Reporter | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.3
Reporter | ||
Updated•24 years ago
|
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Reporter | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.4 → mozilla0.9.5
Reporter | ||
Comment 1•23 years ago
|
||
Note that the float damage stored by the space manager (see bug 86947) must be
cleared each reflow, or perhaps maintained separately from the space manager.
Reporter | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Reporter | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.7 → mozilla0.9.8
Reporter | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.8 → mozilla0.9.9
Reporter | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.9 → Future
Comment 2•22 years ago
|
||
This one has a huge impact on performance (just see the dependency-tree).
Keywords: nsbeta1
Reporter | ||
Comment 3•22 years ago
|
||
Why do you say it's huge? My memory is that it's one of the smaller of the
remaining O(N^2) issues with loading long pages incrementally.
Comment 4•22 years ago
|
||
Well, you know this best. Which are the big killers (bugs) you'd say?
Reporter | ||
Updated•22 years ago
|
Component: Layout → Layout: Floats
Updated•22 years ago
|
Whiteboard: [whitebox]
Updated•22 years ago
|
QA Contact: praveenqa → dsirnapalli
Updated•22 years ago
|
Flags: blocking1.4?
Updated•22 years ago
|
Flags: blocking1.4? → blocking1.4-
Updated•15 years ago
|
QA Contact: dsirnapalli → layout.floats
Comment 6•6 years ago
|
||
Moving to p3 because no activity for at least 2 years.
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Reporter | ||
Updated•4 years ago
|
Assignee: dbaron → nobody
Status: ASSIGNED → NEW
Updated•2 years ago
|
Severity: major → S3
Type: defect → task
Updated•1 month ago
|
Blocks: layoutperfarch
You need to log in
before you can comment on or make changes to this bug.
Description
•