1295649 - telemetry-alert: A11Y_INSTANTIATED_FLAG went from ~5.2% to a flat 0% on or around August 9

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Chris H-C :chutten]

Medusa reported a change in the distribution of A11Y_INSTANTIATED_FLAG[1] on or around August 9.

It went from about 5.2% to a flat 0 [2].

The changelog [3] includes reference to bug 527003.

[1]: http://alerts.telemetry.mozilla.org/index.html#/detectors/1/metrics/791/alerts/?from=2016-08-10&to=2016-08-10
[2]: https://mzl.la/2b9EdD8
[3]: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=720b5d2c84d5b253d4dfde4897e13384dc97a46a&tochange=6cf0089510fad8deb866136f5b92bbced9498447

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

Filed bug 1295651 to add alert emails for future.

Comment 2

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1295649 - adding back removed 'a11y initialized' telemetry measurement.

Review commit: https://reviewboard.mozilla.org/r/72030/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/72030/

Comment 3

[alexander :surkov (:asurkov)]

Comment on attachment 8781646 [details]
Bug 1295649 - adding back removed 'a11y initialized' telemetry measurement.

https://reviewboard.mozilla.org/r/72030/#review69538

::: accessible/base/nsAccessibilityService.cpp:1795
(Diff revision 1)
>      }
>    }
>  
>    MOZ_ASSERT(nsAccessibilityService::gAccessibilityService,
>               "Accessible service is not initialized.");
> +  statistics::A11yInitialized();

I think this should be called when the service is created, r=me with this fixed

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1ab02f408a7d
adding back removed 'a11y initialized' telemetry measurement. r=surkov

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/1ab02f408a7d