Closed
Bug 270136
Opened 20 years ago
Closed 20 years ago
Tp regression from XTF landing
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: bryner, Assigned: bryner)
Details
Attachments
(1 file)
|
3.94 KB,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
Landing of XTF increased Tp by a bit. One culprit was a QI in ConstructFrame that we can avoid by looking at the namespace id. The other culprit seems to be a QI in CreateAnonymousFrames. I believe we can avoid this one as well. I #ifdef'd it out temporarily and the numbers came back down to what they were.
|
Assignee | |
Comment 1•20 years ago
|
||
Reenable the binding parent logic, but move the QI up to the XTF frame construction code.
|
|
Assignee | |
Comment 2•20 years ago
|
||
Comment on attachment 166080 [details] [diff] [review] patch dbaron or bz, can one of you just r+sr this? (or roc, but there's only 2 review fields)
Attachment #166080 -
Flags: superreview?(bzbarsky)
Attachment #166080 -
Flags: review?(dbaron)
|
||
Comment 3•20 years ago
|
||
Comment on attachment 166080 [details] [diff] [review] patch r+sr=bzbarsky
Attachment #166080 -
Flags: superreview?(bzbarsky)
Attachment #166080 -
Flags: superreview+
Attachment #166080 -
Flags: review?(dbaron)
Attachment #166080 -
Flags: review+
|
|
Assignee | |
Comment 4•20 years ago
|
||
checked in.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•