Closed Bug 696076 Opened 10 years ago Closed 10 years ago

Telemetry for Mobile

Categories

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

ARM
Android
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: elan, Assigned: mfinkle)

References

Details

(Keywords: feature, perf, Whiteboard: [birch] [ux needed])

Attachments

(1 file)

We need telemetry for native fennec 1.0.
Whiteboard: [birch] [ux needed]
Assignee: nobody → lmandel
OS: Mac OS X → Android
Hardware: x86 → ARM
Let's just add this as a visible pref. Madhava?
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.
Priority: -- → P4
Taras - Aside from the opt-in UI, what needs to be done to get Telemetry working on mobile?
(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.
Taras - Thanks!

Madhava, Mark - Can we talk about the Telemetry opt-in UI on mobile sometime soon?
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.
Oh, that pref should probably go in the "Privacy and Security" section.
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.
Attached patch patchSplinter Review
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.
Assignee: lmandel → mark.finkle
Attachment #571962 - Flags: review?(kgupta)
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.
Attachment #571962 - Flags: review?(kgupta) → review+
(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
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
These patches were backed while investigating Talos failures.  Now that tests are green again, we will need to reland.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
backout was backed out https://hg.mozilla.org/projects/birch/rev/6f925b45a547
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
20111114041052
http://hg.mozilla.org/projects/birch/rev/859ecdfe0168
Samsung Galaxy SII (Android 2.3.4)
Status: RESOLVED → VERIFIED
Flags: in-litmus?(fennec)
Whiteboard: [birch] [ux needed] → [birch] [ux needed][QA+]
Keywords: feature
Updated start-up test case in litmus:
https://litmus.mozilla.org/show_test.cgi?id=33699
Flags: in-litmus?(fennec) → in-litmus+
Whiteboard: [birch] [ux needed][QA+] → [birch] [ux needed]
Commented wrong bug, please disregard previous comment.
Flags: in-litmus+ → in-litmus?(fennec)
Whiteboard: [birch] [ux needed] → [birch] [ux needed][QA+]
Test case added in litmus under: BFT - options(preferences:
https://litmus.mozilla.org/show_test.cgi?id=50497
Flags: in-litmus?(fennec) → in-litmus+
Whiteboard: [birch] [ux needed][QA+] → [birch] [ux needed]
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.