(dormant account) Assignee
	Description • 13 years ago

      No description provided.

	(dormant account) Assignee
	Comment 1 • 13 years ago

Sid, this is ok with you, right?

	Mike Hommey [:glandium]
	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.

	Sid Stamm [:geekboy or :sstamm]
	Comment 4 • 13 years ago

Since bug 685373 landed, privacy review is resolved. 
https://wiki.mozilla.org/Privacy/Reviews/Telemetry/Default_Browser_Status

	Nathan Froyd [:froydnj]
	Comment 5 • 13 years ago

Assigning to Taras; would be nice to get this landed.

	(dormant account) Assignee
	Comment 6 • 12 years ago

We do not have a clear need for this.