1184190 - basic probe for KidFox usage

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

It would be interesting to know whether this feature will be actively used once released.

Of course the user base for restricted profiles will be really small in comparison to 'normal profiles'. And I assume this is a feature that will be mostly used in the final release version (and not nightly, auroa, beta) where telemetry is not enabled by default. So we should really set our expectations.

Comment 1

[:Margaret Leibovic]

Bug 1183724 will tell us how often Firefox is launched in a guest session. As long as guest session == kidfox, this probe might give us what we need.

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

(In reply to :Margaret Leibovic from comment #1)
> Bug 1183724 will tell us how often Firefox is launched in a guest session.
> As long as guest session == kidfox, this probe might give us what we need.

Guest mode and restricted profiles do share most of the code but they have a different set of restrictions and inGuestMode() will return false for restricted profiles.

Right now we don't even have a public isRestrictedProfile() method. Even RestrictedProfiles.isUserRestricted() will return true for restricted profiles and guest mode, because both have some kind of restrictions.

Comment 3

[Barbara Bermes [:barbara] - NI please!]

I feel this is important before we ship, we want to know how much this is being used, is there also a way to get the time it's in that mode?

Comment 4

[Allison Naaktgeboren :ally]

We can certainly try. This close to the deadline, it's not entirely a given.

Comment 5

[Allison Naaktgeboren :ally]

Barbara, do you have a concrete list of things you'd like to measure?
Usage minutes in this mode might not be technically possible for v1(I'd know when to start the timer...but when to stop it is the tricky question) and I'd definitely need to go through FHR and its approval process for that.

Comment 6

[Barbara Bermes [:barbara] - NI please!]

a) Do people even use it
As long as we can clearly identify how many sessions are in kidfox mode vs. parent/regular mode, that's the most important metric for now.

b) How do people use it
- If possible for later, we should get the timer stuff setup to figure out if the feature is even fully being used (or just tried and then forgotten about).
- What kind of things are being restricted via the parents settings panel to find trends

I'm still waiting for a full flow (from Sam) to understand where other metric check points (e.g. splash screen) are but that shouldn't block you for now.

I hope that clarifies things.

Comment 7

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

Sounds like we should just add to the FENNEC_GECKO_STARTUP_ACTION enum

Comment 8

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: 1184190-kidfox-telemetry.patch

I added a new StartupAction called RESTRICTED. Does this work?

Regarding the first run (bug 1191208): The "Start browsing" button will send "firstrun-browser" too. The "Learn more" link doesn't send anything. Is this okay?

Comment 9

[Sebastian Kaspari (:sebastian; :pocmo)]

(In reply to Barbara Bermes [:bbermes] from comment #6)
> - What kind of things are being restricted via the parents settings panel to
> find trends

What would be the best time to send this? At the beginning of a restricted session? Should we send every restriction with on/off or a list of enabled restrictions (Probably depends on how we want to evaluate these)? We might need to postpone this to 43 (Or try to uplift).

Comment 10

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

Comment on attachment 8644895 [details] [diff] [review]
1184190-kidfox-telemetry.patch

That should work. I made sure the Histogram definition had some empty "bins" so we shouldn't need anything else.

Comment 11

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

(In reply to Sebastian Kaspari (:sebastian) from comment #8)

> The "Learn more" link doesn't send anything. Is this
> okay?

Mostly up to Product. My only thoughts would be: Should a "learn more" probe be Histogram-Telemetry or UI-Telemetry? Given that "firstrun-browser" is UI-Telemetry, I'd lean to doing the same for the link, if needed at all.

Comment 12

[Sebastian Kaspari (:sebastian; :pocmo)]

url:        https://hg.mozilla.org/integration/fx-team/rev/7f0cc9a2bf79950da4f9a4c4ab7df7c73e5a497d
changeset:  7f0cc9a2bf79950da4f9a4c4ab7df7c73e5a497d
user:       Sebastian Kaspari <[e-mail]>
date:       Sun Aug 09 19:58:43 2015 +0200
description:
Bug 1184190 - Send RESTRICTED startup action when restricted profile is used. r=mfinkle

Comment 13

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/7f0cc9a2bf79