1008645 - Add UI telemetry for First-run

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: uitelemetry-firstrun v0.1

This patch adds a "firstrun" Session by using some existing checks in GeckoProfile. We only want "firstrun" for the very first-run and not for any new profile, or even for Guest Browsing (which is also a new profile).

The patch uses the presence of a fully created profiles.ini file to act as the check. There might be other ways, but this seems to work. If there is a better way, please let me know.

If the user chooses to "Clear Data" using the Android "App" settings, we will have a "firstrun" again.

Comment 1

[Brian Nicholson (:bnicholson)]

Comment on attachment 8420610 [details] [diff] [review]
uitelemetry-firstrun v0.1

Review of attachment 8420610 [details] [diff] [review]:
-----------------------------------------------------------------

Nice, this should be useful data.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

https://hg.mozilla.org/integration/fx-team/rev/c2c0ff81cb72

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c2c0ff81cb72

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8420610 [details] [diff] [review]
uitelemetry-firstrun v0.1

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: We want to do before and after testing with telemetry
Testing completed (on m-c, etc.): Works on m-c
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/331db2dde15b

Comment 6

[Flaviu Cos]

Verified as fixed in builds:
32.0a1 (2014-05-27);
31.0a2 (2014-05-27);

Device: Samsung Galaxy Nexus (Android 4.2.1)