Closed Bug 1368121 Opened 6 years ago Closed 6 years ago

Remove telemetry for a11y update time.

Categories

(Core :: Disability Access APIs, enhancement)

Product:
Core
Component:
Disability Access APIs
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: davidb, Assigned: davidb)

References

Details

Attachments

(1 file)

Removal patch
4.12 KB, patch
eeejay
: review+
Details | Diff | Splinter Review 
We can revisit this in the future but right now this probe isn't useful.
Attached patch Removal patch — — Splinter 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: 6 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.