Closed Bug 149378 Opened 18 years ago Closed 12 years ago

nsTableRowGroupFrame and nsTableRowFrame PushState too much

Categories

(Core :: Layout: Tables, defect, P3)

defect

Tracking

()

RESOLVED INVALID
Future

People

(Reporter: dbaron, Assigned: dbaron)

References

Details

(Keywords: perf)

nsTableRowGroupFrame::PaintChildren and nsTableRowFrame::PaintChildren use
PushState/PopState to undo a translation.  It would be faster to just translate
by the negative coordinates.  (This showed up as about 0.15% of a page load
profile on a list of URLs like jrgm's tests.)
Priority: -- → P2
Translate isn't quite the right solution.  See the current code in nsContainerFrame.
Status: NEW → ASSIGNED
Keywords: perf
Priority: P2 → P3
Target Milestone: --- → Future
4510 is blocking this. Why?
Because once that's in we may want to remeasure the perf numbers (especially
once we use that code for standards mode too).
The code has been completely rewritten, I think this bug is obsolete. There is no 
PaintChildren method anymore
the code is gone
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.