Remove telemetry for a11y update time.

RESOLVED FIXED in Firefox 55

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: davidb, Assigned: davidb)

Tracking

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

We can revisit this in the future but right now this probe isn't useful.
Posted patch Removal patchSplinter Review
I think it has been about 71 years since I requested review for a patch I wrote so no idea if I've missed something.
Attachment #8871872 - Flags: review?(eitan)
(In reply to David Bolter [:davidb] from comment #0)
> We can revisit this in the future but right now this probe isn't useful.

can you elaborate it? I'd be curious to see WillRefresh numbers from the wild.
(In reply to alexander :surkov from comment #2)
> (In reply to David Bolter [:davidb] from comment #0)
> > We can revisit this in the future but right now this probe isn't useful.
> 
> can you elaborate it? I'd be curious to see WillRefresh numbers from the
> wild.

I can't find a way to argue that this probe is currently useful. Let me know if you can: https://telemetry.mozilla.org/

Nobody on the team has been looking at it for years AFAIK.
Alex can you find out (or explain) how this is profiler label currently used?
http://searchfox.org/mozilla-central/rev/a14524a72de6f4ff738a5e784970f0730cea03d8/accessible/base/NotificationController.cpp#593
Flags: needinfo?(surkov.alexander)
Comment on attachment 8871872 [details] [diff] [review]
Removal patch

I like this one.
Attachment #8871872 - Flags: review?(eitan) → review+
In the future if we want to measure this in the wild, let's make it more understandable.
Flags: needinfo?(surkov.alexander)
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0fbbdadea65d
Remove telemetry for a11y update time. r=eeejay
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0fbbdadea65d
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.