Closed
Bug 679938
Opened 13 years ago
Closed 12 years ago
report whether firefox is the default browser
Categories
(Toolkit :: Telemetry, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: taras.mozilla, Assigned: taras.mozilla)
References
Details
Attachments
(1 file)
(deleted),
patch
|
glandium
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Updated•13 years ago
|
Keywords: privacy-review-needed
Comment 3•13 years ago
|
||
Comment on attachment 556719 [details] [diff] [review] defaultbrowser simple measurement Review of attachment 556719 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/components/telemetry/TelemetryPing.js @@ +193,4 @@ > var ret = { > // uptime in minutes > + uptime: Math.round((new Date() - si.process) / 60000), > + defaultBrowser: defaultBrowser * 1 I'd use Number(defaultBrowser) instead to make the intent clearer. r=me with the above. As a bonus, I'd add a comma at the end of this line, so that further additions only require adding lines, and not modifying the previous one.
Attachment #556719 -
Flags: review?(mh+mozilla) → review+
Comment 4•13 years ago
|
||
Since bug 685373 landed, privacy review is resolved. https://wiki.mozilla.org/Privacy/Reviews/Telemetry/Default_Browser_Status
Keywords: privacy-review-needed
![]() |
||
Comment 5•13 years ago
|
||
Assigning to Taras; would be nice to get this landed.
Assignee: nobody → taras.mozilla
Assignee | ||
Comment 6•12 years ago
|
||
We do not have a clear need for this.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•