Address performance regressions from frame display lists

RESOLVED FIXED

Status

()

Core
Layout
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: roc, Assigned: roc)

Tracking

({perf, regression})

Trunk
perf, regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

I want to get rid of the space and time penalty from bug 317375. 16K code bloat and Tp up 20ms on luna (10ms on btek).

Updated

12 years ago
Keywords: perf, regression
Created attachment 209751 [details] [diff] [review]
fix Name to be debug-only

First, some trivial cleanup.
Attachment #209751 - Flags: superreview?(dbaron)
Attachment #209751 - Flags: review?(mrbkap)

Updated

12 years ago
Attachment #209751 - Flags: review?(mrbkap) → review+
Attachment #209751 - Flags: superreview?(dbaron) → superreview+
checked that in, thanks.

Comment 3

12 years ago
The fix for bug 324969 seems to have brought btek's Tp back to previous levels. Also see bug 317375 comment 145.
Flags: blocking1.9?
I'm going to close this. Eli's removed a bunch of view code that was obsoleted by frame display lists and I think we just have to believe that in the end it's a net win. Besides.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Flags: blocking1.9?
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.