Closed
Bug 759449
Opened 13 years ago
Closed 13 years ago
Change UI lag telemetry to an EXPONENTIAL histogram
Categories
(Toolkit :: Telemetry, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: bbondy, Assigned: bbondy)
References
Details
Attachments
(1 file, 2 obsolete files)
|
2.49 KB,
patch
|
bbondy
:
review+
|
Details | Diff | Splinter Review |
In Bug 710935 we measure UI lag when it is > 50ms. We initially did this with 200 buckets spanning 30ms. It would be good to increase that range to be higher than 30s and also to use an exponential distribution of buckets instead.
Attachment #628050 -
Flags: review?(taras.mozilla)
|
Assignee | |
Comment 1•13 years ago
|
||
*spanning 30s
|
||
Comment 2•13 years ago
|
||
Comment on attachment 628050 [details] [diff] [review]
Patch v1.
Do we really care about lag over 30s?
Attachment #628050 -
Flags: review?(taras.mozilla) → review+
|
|
||
Comment 3•13 years ago
|
||
Comment on attachment 628050 [details] [diff] [review]
Patch v1.
Also please move (ms) to the tail of the comment. This will eventually get enforced by the build system.
|
|
Assignee | |
Comment 4•13 years ago
|
||
> Do we really care about lag over 30s?
Maybe not but this will only take 1 of the 20 buckets so might as well.
|
|
Assignee | |
Comment 5•13 years ago
|
||
Attachment #628050 -
Attachment is obsolete: true
Attachment #628093 -
Flags: review+
|
|
Assignee | |
Comment 6•13 years ago
|
||
oops you said comment not name, sec updating again.
|
|
Assignee | |
Comment 7•13 years ago
|
||
Attachment #628093 -
Attachment is obsolete: true
Attachment #628095 -
Flags: review+
|
|
Assignee | |
Comment 8•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
You need to log in
before you can comment on or make changes to this bug.
Description
•