Closed
Bug 942054
Opened 11 years ago
Closed 11 years ago
bump up range and increase bucket count for reflow time telemetry metrics
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla28
People
(Reporter: jtd, Assigned: jtd)
Details
(Whiteboard: [qa-])
Attachments
(1 file, 1 obsolete file)
3.39 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
As noted in bug 942049, telemetry data shows a lot of long-running reflows. The data for Firefox 24 shows that 29 million reflows take more than 3 secs. Since a lot of these basically reflect chrome hangs, I propose we bump up the max value for these metrics and increase the number of buckets: HTML_FOREGROUND_REFLOW_MS HTML_BACKGROUND_REFLOW_MS Might be interesting to log additional metrics *when* the reflow time is above a given threshold, but it's probably a good thing to bump these ASAP.
Comment 1•11 years ago
|
||
Don't forget to rename the histograms when you change the parameters.
Assignee | ||
Comment 2•11 years ago
|
||
Add more buckets and extend the range for content reflow metrics. Also add a new boolean metric, LONG_REFLOW_INTERRUPTIBLE, which tracks whether long reflows were interruptible or not. The XUL reflow times don't have very many long running reflows so I didn't do the same for them.
Attachment #8337501 -
Flags: review?(bzbarsky)
Assignee | ||
Comment 3•11 years ago
|
||
Bump the names of the updated metrics to "xxx_2".
Attachment #8337505 -
Flags: review?(bzbarsky)
Assignee | ||
Updated•11 years ago
|
Attachment #8337501 -
Attachment is obsolete: true
Attachment #8337501 -
Flags: review?(bzbarsky)
Comment 4•11 years ago
|
||
Comment on attachment 8337505 [details] [diff] [review] patch, bump range and bucket count for content reflows r=me. Thank you!
Attachment #8337505 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 5•11 years ago
|
||
Pushed to inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/555b1691b0a4
Comment 6•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/555b1691b0a4
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Updated•10 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•