Closed
Bug 311592
Opened 19 years ago
Closed 18 years ago
Toggling backgrounds on lots of abs pos divs is O(N^2)
Categories
(Core :: Web Painting, defect)
Core
Web Painting
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: bzbarsky, Assigned: roc)
References
(Depends on 2 open bugs, )
Details
(Keywords: perf)
The first time we set backgrounds on the divs takes O(N^2) time, since we call
nsViewManager::SetViewContentTransparency on each one, which does a
IsViewInserted, which is O(N) in number of previous siblings.
Perhaps we should store the inserted bit on the view itself or something?
|
Assignee | |
Comment 1•19 years ago
|
||
We should, yes. But a) we won't be fixing this on branch and b) on trunk I want
the entire view tree to go away.
|
Reporter | |
Comment 2•19 years ago
|
||
Makes sense. I'm just making sure I get bugs filed on all the stuff that pops
up as I profile the tests in bug 297959.
Isn't this bug 233441 ?
|
|
Reporter | |
Comment 4•19 years ago
|
||
Dependent, not same. We could fix this bug in other ways, possibly.
Depends on: 233441
|
||
Comment 5•18 years ago
|
||
Is this still a problem? SetViewContentTransparency was removed in Bug 370379, and abs. pos. divs no longer create views.
|
|
Reporter | |
Comment 6•18 years ago
|
||
Yeah, this is no longer an issue.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
|
||
Updated•6 years ago
|
Component: Layout: View Rendering → Layout: Web Painting
You need to log in
before you can comment on or make changes to this bug.
Description
•