Extend tab closing and switching Telemetry probes

RESOLVED FIXED in Firefox 59

Status

()

enhancement
RESOLVED FIXED
a year ago
8 months ago

People

(Reporter: mconley, Assigned: mconley)

Tracking

unspecified
Firefox 59
Points:
---

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(1 attachment)

The following probes are set to expire on 2018-01-22, and are still important in our measurement of tab closing and switching performance:

FX_TAB_CLOSE_PERMIT_UNLOAD_TIME_M
FX_TAB_CLOSE_TIME_ANIM_MS
FX_TAB_CLOSE_TIME_NO_ANIM_MS
FX_TAB_SWITCH_TOTAL_E10S_MS
TABCHILD_PAINT_TIME

I'd like to kick the can and extend them another set of versions.
(Assignee)

Comment 2

a year ago
Hey liuche, are you willing to sign-off on this with a data-review+?
Flags: needinfo?(liuche)

Comment 3

a year ago
mozreview-review
Comment on attachment 8939569 [details]
Bug 1427744 - Extend the lifetime of Telemetry probes that measure tab close and switch performance. , data-review=liuche

https://reviewboard.mozilla.org/r/209882/#review215448

rs=me, but do we ever want these to expire? Maybe we should just set them to expires=never?

(AIUI you will still need data/privacy-review to extend the probes here)
Attachment #8939569 - Flags: review?(gijskruitbosch+bugs) → review+
(Assignee)

Comment 4

a year ago
(In reply to :Gijs from comment #3)
> rs=me, but do we ever want these to expire? Maybe we should just set them to
> expires=never?
> 

I could be wrong on this, but I seem to recall that "expires never" is discouraged because we might then forget about the probes and collect forever without thinking about whether we need the data anymore. This kinda "renewal" model means we have to consider it every once in a while. I'm not 100% sure on the policy, however.

> (AIUI you will still need data/privacy-review to extend the probes here)

Yep! liuche needinfo'd. :)
Assignee: nobody → mconley
Comment on attachment 8939569 [details]
Bug 1427744 - Extend the lifetime of Telemetry probes that measure tab close and switch performance. , data-review=liuche

Try confirms that this fixes the issues reported in bug 1428081.
Attachment #8939569 - Flags: review?(liuche)
Attachment #8939569 - Flags: feedback+
Comment on attachment 8939569 [details]
Bug 1427744 - Extend the lifetime of Telemetry probes that measure tab close and switch performance. , data-review=liuche

Yes! Approving this extension of the probe.
Flags: needinfo?(liuche)
Attachment #8939569 - Flags: review?(liuche) → review+

Comment 7

a year ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5b318239826f
Extend the lifetime of Telemetry probes that measure tab close and switch performance. r=Gijs, data-review=liuche

Comment 8

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5b318239826f
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
See Also: → 1488952
You need to log in before you can comment on or make changes to this bug.