Closed
Bug 790379
Opened 12 years ago
Closed 12 years ago
add more profile labels for style change processing
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla18
People
(Reporter: dbaron, Assigned: dbaron)
Details
Attachments
(2 files)
2.75 KB,
patch
|
cjones
:
review+
|
Details | Diff | Splinter Review |
2.88 KB,
patch
|
cjones
:
review+
|
Details | Diff | Splinter Review |
See bug 780341 and bug 779968 for background. Looking at profiles for style change processing is somewhat confusing right now; the stuff currently inside the CSS::ProcessRestyles label is really two different things, recomputation of style data and then handling the resulting changes. This patch adds two labels to split it into those major parts. It's untested.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #660194 -
Flags: review?(jones.chris.g)
Comment on attachment 660194 [details] [diff] [review] Add more profile labels for the separate parts of restyle processing. () rs=me
Attachment #660194 -
Flags: review?(jones.chris.g) → review+
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/557733e500ca
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/557733e500ca
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Assignee | ||
Comment 5•12 years ago
|
||
Two more labels, to try to distinguish benefits we'd get from bug 780341 comment 30 (would help with the time inside FileRules) vs. benefits we'd get from bug 780341 comment 38 (would help with the time inside CalcStyleDifference). With these labels, I'm a bit concerned that they're too far inside the loop and might influence the timing. However, I *think* they're going to occur the same number of times in the cases we're looking at, since we won't actually end up with the same style context in any of the cases where we hit FileRules.
Attachment #660585 -
Flags: review?(jones.chris.g)
Comment on attachment 660585 [details] [diff] [review] two more labels rs=me (rubberstamp because I don't know this code and am trusting you to put them in the right place.) You're more than welcome to land any more labels you want to add as rs=me, too.
Attachment #660585 -
Flags: review?(jones.chris.g) → review+
Assignee | ||
Comment 7•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f2172a4fab4c
Comment 8•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f2172a4fab4c
You need to log in
before you can comment on or make changes to this bug.
Description
•