1470956 - Remove the E10S_STATUS telemetry probe

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement)

Description

[:Felipe Gomes (needinfo for replies!)]

This probe was used during the e10s rollout, but at this point no one is looking at it anymore.

Comment 1

[:Felipe Gomes (needinfo for replies!)]

Attachment: Bug 1470956 - Remove the E10S_STATUS telemetry probe.

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

Comment 2

[Chris H-C :chutten]

Comment on attachment 8987585 [details]
Bug 1470956 - Remove the E10S_STATUS telemetry probe.

https://reviewboard.mozilla.org/r/252826/#review259340

Two nits, then it's good to go.

::: commit-message-b2919:1
(Diff revision 1)
> +Bug 1470956 - Remove the E10S_STATUS telemetry probe. r=chutten

Could you also mention E10S_BLOCKED_FROM_RUNNING in the commit message somewhere?

::: toolkit/components/telemetry/histogram-whitelists.json:653
(Diff revision 1)
>      "DNT_USAGE",
>      "DWRITEFONT_DELAYEDINITFONTLIST_COLLECT",
>      "DWRITEFONT_DELAYEDINITFONTLIST_COUNT",
>      "DWRITEFONT_DELAYEDINITFONTLIST_TOTAL",
>      "DWRITEFONT_INIT_PROBLEM",
>      "E10S_BLOCKED_FROM_RUNNING",

Needs to be removed from here, too

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e7dd48ffda9d
Remove the E10S_STATUS and E10S_BLOCKED_FROM_RUNNING telemetry probes. r=chutten

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/e7dd48ffda9d