Closed
Bug 1358620
Opened 8 years ago
Closed 8 years ago
Add telemetry for non-MPC extensions
Categories
(Toolkit :: Add-ons Manager, enhancement, P1)
Toolkit
Add-ons Manager
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: aswan, Assigned: aswan)
References
Details
(Whiteboard: triaged)
Attachments
(1 file)
In bug 1352204 we will be disabling non-MPC extensions by default on Nightly.
This bug is to add a new boolean scalar to telemetry indicating whether a given profile has any non-MPC extensions installed. This will serve two purposes:
1. Allow us to quantify how many users were affected by the non-MPC change (more specifically, how many were affected enough to flip the pref back)
2. Allow people studying performance data to optionally filter out data from users who have non-MPC extensions enabled, when it is known that those extensions introduce unwanted noise into performance related metrics.
Assignee | ||
Comment 1•8 years ago
|
||
I think this can get done quickly but a couple of questions:
1. Any objection to adding extensions.allow-non-mpc-extensions to DEFAULT_ENVIRONMENT_PREFS too?
2. I can either add a new boolean property to the "addons" part of the environment indicating whether any non-MPC extensions are installed, or add a multiprocessCompatible property to each individual extension. Any reason to prefer one of those over the other?
Assignee: nobody → aswan
Flags: needinfo?(benjamin)
Comment 2•8 years ago
|
||
> 1. Any objection to adding extensions.allow-non-mpc-extensions to
> DEFAULT_ENVIRONMENT_PREFS too?
No please do that.
> 2. I can either add a new boolean property to the "addons" part of the
> environment indicating whether any non-MPC extensions are installed, or add
> a multiprocessCompatible property to each individual extension. Any reason
> to prefer one of those over the other?
I slightly prefer the .multiprocessCompatible property because it's more flexible for measuring other things such as "which MPC=false addons are most popular on channel X".
Flags: needinfo?(benjamin)
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Attachment #8861125 -
Flags: review?(benjamin)
Comment 4•8 years ago
|
||
mozreview-review |
Comment on attachment 8861125 [details]
Bug 1358620 Add telemetry for non-MPC extensions
https://reviewboard.mozilla.org/r/133084/#review136302
::: toolkit/components/telemetry/TelemetryEnvironment.jsm:630
(Diff revision 1)
> updateDay: Utils.millisecondsToDays(updateDate.getTime()),
> signedState: addon.signedState,
> isSystem: addon.isSystem,
> isWebExtension: addon.isWebExtension,
> + multiprocessCompatible: Boolean(addon.multiprocessCompatible),
> };
This needs to be documented in environment.rst also.
Attachment #8861125 -
Flags: review?(benjamin) → review-
Comment hidden (mozreview-request) |
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8861125 [details]
Bug 1358620 Add telemetry for non-MPC extensions
https://reviewboard.mozilla.org/r/133084/#review136314
code review and data-review=me
Please ping somebody on the data team (gfritzsche?) about getting this into the ping schema->derived data sets->longitudinal
Attachment #8861125 -
Flags: review?(benjamin) → review+
Updated•8 years ago
|
Priority: -- → P1
Whiteboard: triaged
Assignee | ||
Comment 7•8 years ago
|
||
Georg, re comment 6, is that something I should just open a new bug for or do you have some other process?
Flags: needinfo?(gfritzsche)
Comment hidden (mozreview-request) |
Pushed by aswan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4bac91d15349
Add telemetry for non-MPC extensions r=bsmedberg
Comment 10•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Comment 11•8 years ago
|
||
Mark, can you take a look at what actions are needed per comment 6?
This adds to the environment:
- an entry to userPrefs
- an addon property
Flags: needinfo?(gfritzsche) → needinfo?(mreid)
Comment 12•8 years ago
|
||
I've filed bug 1360174 to add this data to the Longitudinal dataset, and bug 1360177 to add it to Main Summary and downstream Addon-related datasets.
Flags: needinfo?(mreid)
You need to log in
before you can comment on or make changes to this bug.
Description
•