Closed
Bug 726007
Opened 13 years ago
Closed 13 years ago
Add telemetry probes for forgetSkippable
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: smaug, Assigned: smaug)
References
Details
Attachments
(2 files, 1 obsolete file)
|
3.47 KB,
patch
|
Details | Diff | Splinter Review | |
|
3.88 KB,
patch
|
mccr8
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
||
Comment 1•13 years ago
|
||
So, I wonder what the best thing to do here is. The most direct thing is to record every forgetSkippable. We could also only record those above a certain length of time.
|
Assignee | |
Comment 2•13 years ago
|
||
I'll upload -w patch in a second
Attachment #596186 -
Flags: review?(continuation)
|
|
Assignee | |
Comment 3•13 years ago
|
||
|
|
Assignee | |
Comment 4•13 years ago
|
||
PRTime startTime = PR_Now(); is one line.
Attachment #596188 -
Flags: review?(continuation)
|
|
Assignee | |
Updated•13 years ago
|
Attachment #596186 -
Attachment is obsolete: true
Attachment #596186 -
Flags: review?(continuation)
|
|
||
Comment 5•13 years ago
|
||
Comment on attachment 596188 [details] [diff] [review]
patch
Review of attachment 596188 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks!
Attachment #596188 -
Flags: review?(continuation) → review+
|
|
Assignee | |
Comment 6•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•