We need to create a telemetry histogram for UPDATE_SERVICE_INSTALLED. It should be a boolean histogram with values 0, 1 which correspond with whether or not the MozillaMaintenance service exists on the machine.
This telemetry ping should only be sent from Windows machines.
Created attachment 631708 [details] [diff] [review] Patch v2 -Tested
After I wrote this patch I realized we have a registry entry for Installed with a DWORD value of 1 or 0. But this patch may be more deterministic. But I'm open to changing it if you prefer to the reg way.
Created attachment 633125 [details] [diff] [review] Patch v3 After thinking about it more, our other checks use the Installed reg value already so we might as well stay consistent. Doing a reg check is cleaner than using ctypes anyway. The code is probably fine either way but in theory the SCM can be locked and it might take longer to check via the SCM. Also this value will always be in sync with whether or not the service is actually installed unless a user goes and modifies it themselves.
Attachment #633125 - Flags: review?(robert.bugzilla)
6 years ago
Attachment #633125 - Flags: review?(robert.bugzilla) → review+
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.