1558900 - Consider removing PLUGINS_NOTIFICATION_USER_ACTION_2 telemetry histogram given we have POPUP_NOTIFICATION_STATS

Status: RESOLVED FIXED

(Firefox :: Site Identity, task, P3)

Description

[:Gijs (he/him)]

Given https://phabricator.services.mozilla.com/D34215#inline-205081, it seems we may be able to remove this histogram. Johann, do you know who could make this call?

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug is a task, but please change it back in case of error.

Comment 2

[Johann Hofmann [:johannh]]

I dunno, who's in charge of plugins nowadays? Jimm? Cpeterson?

If we can't identify anyone who still needs it I'll make the call and say we should remove it.

Comment 3

[Chris Peterson [:cpeterson]]

who's in charge of plugins nowadays? Jimm? Cpeterson?

jimm is the plugins module owner.

Does POPUP_NOTIFICATION_STATS report when which door hanger is being interacted with? We would like to know how often the "Allow Adobe Flash" door hanger is shown and how often the user chooses to activate it.

Comment 4

[Johann Hofmann [:johannh]]

(In reply to Chris Peterson [:cpeterson] from comment #3)

who's in charge of plugins nowadays? Jimm? Cpeterson?

jimm is the plugins module owner.

Does POPUP_NOTIFICATION_STATS report when which door hanger is being interacted with? We would like to know how often the "Allow Adobe Flash" door hanger is shown and how often the user chooses to activate it.

Ok, well, if you are currently using this probe for data measurement it's probably a good idea to keep using it to avoid surprises. I was mostly thinking that it might be unused.

POPUP_NOTIFICATION_STATS does report which doorhanger was interacted with, yes.

Comment 5

[Jim Mathies [:jimm]]

Lets keep this around in case it's in data reports or queries. We'll be able to remove all the plugin stuff next spring.

Comment 6

[Gregory Pappas [:gregp]]

Attachment: Bug 1558900 - Remove PLUGINS_NOTIFICATION_USER_ACTION_2 and PLUGINS_NOTIFICATION_SHOWN telemetry histograms r?Gijs!

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/73c167fba846
Remove PLUGINS_NOTIFICATION_USER_ACTION_2  and PLUGINS_NOTIFICATION_SHOWN telemetry histograms r=Gijs

Comment 8

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/73c167fba846