696076 - Telemetry for Mobile

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P4)

Description

[Erin Lancaster [:elan]]

We need telemetry for native fennec 1.0.

Comment 1

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

Let's just add this as a visible pref. Madhava?

Comment 2

[Lawrence Mandel [:lmandel] (use needinfo)]

While thinking about the pref please keep in mind that we want to encourage people to enable/opt-in to Telemetry. I think we need to do better than having a pref that has to be discovered by the user.

Comment 3

[Lawrence Mandel [:lmandel] (use needinfo)]

Taras - Aside from the opt-in UI, what needs to be done to get Telemetry working on mobile?

Comment 4

[(dormant account)]

(In reply to Lawrence Mandel [:lmandel] from comment #3)
> Taras - Aside from the opt-in UI, what needs to be done to get Telemetry
> working on mobile?

That's all.

Comment 5

[Lawrence Mandel [:lmandel] (use needinfo)]

Taras - Thanks!

Madhava, Mark - Can we talk about the Telemetry opt-in UI on mobile sometime soon?

Comment 6

[Madhava Enros [:madhava]]

So we can certainly have a pref for this, given how important it is. What would be a good, short, non-jargony explanation we can present to people?

Off the top of my head:

Main pref title:  Send performance data
Secondary line:   Your anonymous data helps make Firefox faster

I'd be inclined to just have it be on by default in Nightly and Aurora (even beta?)

There's the separate issue about getting people to opt in on final.

Comment 7

[Madhava Enros [:madhava]]

Oh, that pref should probably go in the "Privacy and Security" section.

Comment 8

[Lawrence Mandel [:lmandel] (use needinfo)]

There are two separate issues here:

1. We need to expose the ability to enable/disable Telemetry in the user preferences.

2. We need to advertise Telemetry to our users and encourage them to opt-in.

I suggest that we tackle #1 in this bug as within #1 we have no Telemetry on mobile. I have opened bug 699513 to track #2.

Comment 9

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

Attachment: patch

Simple patch to add a preference to the Privacy section. I did not add the secondary description since we don't do it anywhere else. I can make a followup patch if needed.

Comment 10

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 571962 [details] [diff] [review]
patch

Not sure "Send performance data" is really that good of a string on it's own, but we can change that later.

Comment 11

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

(In reply to Kartikaya Gupta (:kats) from comment #10)
> Comment on attachment 571962 [details] [diff] [review] [diff] [details] [review]
> patch
> 
> Not sure "Send performance data" is really that good of a string on it's
> own, but we can change that later.

We use that string in XUL Fennec and the opt-in prompt will give more context too.

https://hg.mozilla.org/projects/birch/rev/6eeeae97b14d

Comment 12

[Wesley Johnston (:wesj)]

These patches were backed while investigating Talos failures.  Now that tests are green again, we will need to reland.

Comment 13

[Brad Lassey [:blassey] (use needinfo?)]

backout was backed out https://hg.mozilla.org/projects/birch/rev/6f925b45a547

Comment 14

[Aaron Train [:aaronmt]]

20111114041052
http://hg.mozilla.org/projects/birch/rev/859ecdfe0168
Samsung Galaxy SII (Android 2.3.4)

Comment 15

[Camelia Urian]

Updated start-up test case in litmus:
https://litmus.mozilla.org/show_test.cgi?id=33699

Comment 16

[Camelia Urian]

Commented wrong bug, please disregard previous comment.

Comment 17

[Camelia Urian]

Test case added in litmus under: BFT - options(preferences:
https://litmus.mozilla.org/show_test.cgi?id=50497