Closed
Bug 763016
Opened 12 years ago
Closed 12 years ago
Report telemetry data for if the MozillaMaintenance service is installed
Categories
(Toolkit :: Application Update, defect)
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: bbondy, Assigned: bbondy)
References
Details
Attachments
(1 file, 2 obsolete files)
3.90 KB,
patch
|
robert.strong.bugs
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•12 years ago
|
Target Milestone: --- → mozilla16
Assignee | ||
Comment 1•12 years ago
|
||
This telemetry ping should only be sent from Windows machines.
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #631603 -
Attachment is obsolete: true
Attachment #631708 -
Flags: review?(robert.bugzilla)
Assignee | ||
Comment 4•12 years ago
|
||
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.
Assignee | ||
Comment 5•12 years ago
|
||
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)
Assignee | ||
Updated•12 years ago
|
Attachment #631708 -
Attachment is obsolete: true
Attachment #631708 -
Flags: review?(robert.bugzilla)
Updated•12 years ago
|
Attachment #633125 -
Flags: review?(robert.bugzilla) → review+
Assignee | ||
Comment 6•12 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/1e497edb48c1
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•