Closed
Bug 1007264
Opened 10 years ago
Closed 10 years ago
Opt-in message is not clear that stumbling happens when Fennec is not running
Categories
(Android Background Services Graveyard :: Geolocation, defect)
Android Background Services Graveyard
Geolocation
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 35
People
(Reporter: ggp, Assigned: garvan)
References
Details
(Whiteboard: geolocation, mozilla location services)
Attachments
(1 file, 3 obsolete files)
4.68 KB,
patch
|
rnewman
:
review+
|
Details | Diff | Splinter Review |
Do we want to have Fennec passively collecting cell, location and wifi data while it is not running? For example, say the user has opted-in to data collection, then reboots their phone, so Fennec is decidedly not running unless the user starts it again. Should we still be listening passively for locations after the reboot, but before Fennec is opened for the next time? While that behavior is technically feasible, it could be considered invasive by the user. Note that our current opt-in text: "Receives Wi-Fi and cellular location data when running in the background and shares it with &vendorShortName; to improve our geolocation service" specifies that data collection will happen while Fennec is running in the background, but says nothing about when it is not running at all. So in my opinion, if we are to scan passively at all times, we should at least consider updating the opt-in text to make that clear.
Reporter | ||
Updated•10 years ago
|
Flags: needinfo?(udevi)
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Reporter | ||
Comment 4•10 years ago
|
||
(In reply to Chris Peterson (:cpeterson) from comment #3) > I don't think users would (be able to) make such a subtle distinction > between Fennec as a running foreground app, a paused background app, a > background app that was killed by memory pressure, and having Fennec > installed but not launched since last rebooting their phone. This is a very good point, but I worry that Android users may have _some_ simpler notion of "running" vs. "not running" (conveyed by the system's UI), and that our opt-in text may not reflect that notion. Personally, I think considering that an app is "not running" before it is launched is not such a subtle distinction, but I admittedly know nothing about the expectations of most of our users, so I can absolutely be convinced otherwise. Ultimately, of course, the goal is not to force the user to distinguish between very technical and subtle cases, but rather to make sure our collection policy and opt-in text match whatever simpler notion of "running" most users would have.
Comment 5•10 years ago
|
||
As you suggested earlier, I think we can avoid this ambiguity by just removing the phrase "when running" from our opt-in test. Your other concern was about the phrase "Receiving ... data", which we could change to something like "Collect". So the current text: > "Receives Wi-Fi and cellular location data when running in the background and > shares it with &vendorShortName; to improve our geolocation service" could become something like: > "Collects Wi-Fi and cellular location data and > shares it with &vendorShortName; to improve our geolocation service"
Summary: Should we collect data while Fennec is NOT running? → [geolocation-stumbler] Opt-in message is not clear that stumbling happens when not running
Is this bug referring to this notice: https://bug903535.bugzilla.mozilla.org/attachment.cgi?id=8403028 If yes, a few questions: (1) What is the concern between the word "receives" and "collects"? (2) Is the business decision to change the collection so that, for users who opt-into reporting location on their Android devices, Mozilla will receive location data regardless of whether Firefox is active or terminated? - If yes, I will change the language to reflect this. However, wouldn't many people choose not to opt-into this collection because of the batter drain issue? (3) I don't know how many people opted into reporting location, but if it's a small number, it seems likely that those are the same people who also have MozStumbler already running. Is this a double instance of reporting that makes the battery drain worse? If so, seems like not much benefit to extending the Fennec data collection.
> However, wouldn't > many people choose not to opt-into this collection because of the batter > drain issue? The implementation should have little measurable power impact. We use passive listening to location, so if another app requests a GPS location, we get notified, and get that location. > (3) I don't know how many people opted into reporting location, but if it's > a small number, it seems likely that those are the same people who also have > MozStumbler already running. Is this a double instance of reporting that > makes the battery drain worse? If so, seems like not much benefit to > extending the Fennec data collection. See above regarding power impact. Hopefully with a better UX (bug 1027078), we can sell folks the idea there is a no-cost way to contribute to an open location database, just click the ok button and we will do the rest in the background.
Reporter | ||
Comment 8•10 years ago
|
||
(In reply to Mika from comment #6) > (1) What is the concern between the word "receives" and "collects"? I think "receives" feels too passive to accurately describe what we're doing, and suggests a couple of questions that are left unanswered -- "receives" from whom? What can I do to stop whoever is "sending" the location data from doing so? "Collects" sounds better to me because it clearly states that we're actively gathering location data, which will otherwise not be shared with anyone if the user doesn't opt-in. Of course, this distinction comes from my personal interpretation of the text, so if it looks inconsequential to you, feel free to stick with "receives".
Updated•10 years ago
|
Component: General → Geolocation
Product: Firefox for Android → Android Background Services
Summary: [geolocation-stumbler] Opt-in message is not clear that stumbling happens when not running → Opt-in message is not clear that stumbling happens when Fennec is not running
Comment 9•10 years ago
|
||
Mika: what are the next steps here? The current text is: "Receives Wi-Fi and cellular location data when running in the background and shares it with Mozilla to improve our geolocation service" If we replace the word "receives" with "collects" and remove the phrase "when running" to (perhaps too implicitly?) allow background data collection, would the following text be acceptable? "Collects Wi-Fi and cellular location data and shares it with Mozilla to improve our geolocation service"
Flags: needinfo?(udevi)
Comment 10•10 years ago
|
||
Does this language address the concerns: The title is: Mozilla Location Service The old text is here: http://screencast.com/t/dNVtbCxlMf The new text would read: "Receives Wi-Fi and cellular location data based on the approximate location of your device to improve Mozilla's geolocation service" I prefer to use the word "receive" instead of "collect" because Mozilla receives data when users enable the option. This is how we describe our data practice in several other notices.
Flags: needinfo?(udevi)
Comment 11•10 years ago
|
||
I see how "receive" is more technically accurate than "collect", but your proposed text does not mention that Fennec will upload this data to Mozilla. Copying the "Telemetry" setting's description in your screenshot, how about something like the following? "Shares Wi-Fi and cellular location data based on the approximate location of your device to improve Mozilla's geolocation service"
Comment 12•10 years ago
|
||
Since it's called "Mozilla Location Service" I think it's clear that the data is received by Mozilla. But if you want to be more clear, here's a slight tweak of your proposal: "Shares Wi-Fi and cellular location data based on the approximate location of your device with Mozilla to improve our geolocation service"
Comment hidden (typo) |
Comment 14•10 years ago
|
||
I like it. :) Now we just need Garvan's feedback.
Comment 15•10 years ago
|
||
Yay!
Assignee | ||
Comment 16•10 years ago
|
||
looks good, thanks
Assignee | ||
Comment 17•10 years ago
|
||
Oops, almost forgot to update this message. Here is the patch. Nick can you suggest what the process is for changing strings? Who reviews, who does it go to next for localization, those types of things.
Attachment #8499733 -
Flags: feedback?(nalexander)
Comment 18•10 years ago
|
||
Comment on attachment 8499733 [details] [diff] [review] bug1007264-optin-wording.patch Review of attachment 8499733 [details] [diff] [review]: ----------------------------------------------------------------- As for l10n review, if you have the sign-off from legal, no further code review necessary; but I'll flag Jeff since this is Android specific and legal-sensitive. ::: mobile/android/base/locales/en-US/android_strings.dtd @@ +265,5 @@ > <!-- Localization note (datareporting_crashreporter_title_short) : This string matches > (crashReporterSection.label) in en-US/chrome/browser/preferences/advanced.dtd.--> > <!ENTITY datareporting_crashreporter_title_short "Crash Reporter"> > <!ENTITY datareporting_wifi_title2 "&vendorShortName; Location Service"> > +<!ENTITY datareporting_wifi_geolocation_summary "Shares Wi-Fi and cellular location data based on the approximate location of your device with &vendorShortName; to improve our geolocation service"> You must rename the entity (so make it ..._summary2, like the one above) when you change a string. You'll need to update strings.xml.in to refer to it as well. nit: geolocation service*s* -- we may have more than one, right?
Attachment #8499733 -
Flags: feedback?(nalexander)
Attachment #8499733 -
Flags: feedback?(jbeatty)
Attachment #8499733 -
Flags: feedback+
Comment 19•10 years ago
|
||
Comment on attachment 8499733 [details] [diff] [review] bug1007264-optin-wording.patch Is this being uplifted? If not, the change will hit Aurora in a week and will be available for all official locales to translate. I don't think this requires any additional explanation in the localization note or in the entity beyond changing it to summary2, as what was already mentioned. Looks good to me.
Attachment #8499733 -
Flags: feedback?(jbeatty) → feedback+
Assignee | ||
Comment 20•10 years ago
|
||
(In reply to Nick Alexander :nalexander from comment #18) > Comment on attachment 8499733 [details] [diff] [review] > bug1007264-optin-wording.patch > > Review of attachment 8499733 [details] [diff] [review]: > ----------------------------------------------------------------- > > As for l10n review, if you have the sign-off from legal, no further code > review necessary; but I'll flag Jeff since this is Android specific and > legal-sensitive. > > ::: mobile/android/base/locales/en-US/android_strings.dtd > @@ +265,5 @@ > > <!-- Localization note (datareporting_crashreporter_title_short) : This string matches > > (crashReporterSection.label) in en-US/chrome/browser/preferences/advanced.dtd.--> > > <!ENTITY datareporting_crashreporter_title_short "Crash Reporter"> > > <!ENTITY datareporting_wifi_title2 "&vendorShortName; Location Service"> > > +<!ENTITY datareporting_wifi_geolocation_summary "Shares Wi-Fi and cellular location data based on the approximate location of your device with &vendorShortName; to improve our geolocation service"> > > You must rename the entity (so make it ..._summary2, like the one above) > when you change a string. You'll need to update strings.xml.in to refer to > it as well. > > nit: geolocation service*s* -- we may have more than one, right? Just one service that I can think of (it does have both a submission and query interface).
Assignee | ||
Comment 21•10 years ago
|
||
Thanks for the review Nick, hopefully I did this correctly: - Changed to datareporting_wifi_geolocation_summary2 - updated mobile/android/base/strings.xml.in to match
Attachment #8499733 -
Attachment is obsolete: true
Attachment #8499779 -
Flags: review?(nalexander)
Comment 22•10 years ago
|
||
Comment on attachment 8499779 [details] [diff] [review] bug1007264-optin-wording.patch Review of attachment 8499779 [details] [diff] [review]: ----------------------------------------------------------------- Yep, lgtm.
Attachment #8499779 -
Flags: review?(nalexander) → review+
Assignee | ||
Comment 23•10 years ago
|
||
> Is this being uplifted?
Just for landing in Nightly and following the train. This wording change is to correspond with how data collection has already changed in Nightly (35).
Keywords: checkin-needed
Comment 24•10 years ago
|
||
(In reply to Garvan Keeley [:garvank] from comment #23) > > Is this being uplifted? > > Just for landing in Nightly and following the train. This wording change is > to correspond with how data collection has already changed in Nightly (35). Sounds great!
Assignee | ||
Comment 25•10 years ago
|
||
running a try+tests before checkin needed
Keywords: checkin-needed
Assignee | ||
Comment 26•10 years ago
|
||
try green https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=6bbe9bf3c7d5
Keywords: checkin-needed
Comment 27•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/29fcac263c20
Assignee: nobody → gkeeley
Keywords: checkin-needed
Whiteboard: geolocation, mozilla location services → geolocation, mozilla location services[fixed-in-fx-team]
Comment 28•10 years ago
|
||
Backed out for the robocop failures noted in bug 1061991 comment 7. Which were also visible on the Try run. Please verify that your Try pushes are green before requesting checkin in the future. https://hg.mozilla.org/integration/fx-team/rev/d99a7f64432a
Whiteboard: geolocation, mozilla location services[fixed-in-fx-team] → geolocation, mozilla location services
Assignee | ||
Comment 29•10 years ago
|
||
Crap. Sorry. Friday night impatience. Actually passes robocop#4 -for real- on both platforms: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=7ca831b0c2ca
Attachment #8499779 -
Attachment is obsolete: true
Attachment #8500803 -
Flags: review?(nalexander)
Assignee | ||
Comment 30•10 years ago
|
||
Comment on attachment 8500803 [details] [diff] [review] bug1007264-optin-wording.patch Review of attachment 8500803 [details] [diff] [review]: ----------------------------------------------------------------- ::: mobile/android/base/tests/testSettingsMenuItems.java @@ +171,5 @@ > } > > if (AppConstants.MOZ_STUMBLER_BUILD_TIME_ENABLED) { > // Anonymous cell tower/wifi collection > + String[] networkReportingUi = { "Mozilla Location Service", "Shares Wi-Fi and cellular location data based on the approximate location of your device with Mozilla to improve our geolocation service" }; This was not updated in the first patch, hence Robocop test failure.
Comment 31•10 years ago
|
||
Comment on attachment 8500803 [details] [diff] [review] bug1007264-optin-wording.patch Review of attachment 8500803 [details] [diff] [review]: ----------------------------------------------------------------- r+ for the test fix, and a suggestion for a smoother sentence. ::: mobile/android/base/locales/en-US/android_strings.dtd @@ +265,5 @@ > <!-- Localization note (datareporting_crashreporter_title_short) : This string matches > (crashReporterSection.label) in en-US/chrome/browser/preferences/advanced.dtd.--> > <!ENTITY datareporting_crashreporter_title_short "Crash Reporter"> > <!ENTITY datareporting_wifi_title2 "&vendorShortName; Location Service"> > +<!ENTITY datareporting_wifi_geolocation_summary2 "Shares Wi-Fi and cellular location data based on the approximate location of your device with &vendorShortName; to improve our geolocation service"> This is a pretty tortured sentence. How about Shares approximate Wi-Fi and cellular location data with &vendorShortName; to improve our geolocation service ::: mobile/android/base/tests/testSettingsMenuItems.java @@ +171,5 @@ > } > > if (AppConstants.MOZ_STUMBLER_BUILD_TIME_ENABLED) { > // Anonymous cell tower/wifi collection > + String[] networkReportingUi = { "Mozilla Location Service", "Shares Wi-Fi and cellular location data based on the approximate location of your device with Mozilla to improve our geolocation service" }; This should really use string substitution, but oh well.
Attachment #8500803 -
Flags: review?(nalexander) → review+
Assignee | ||
Comment 32•10 years ago
|
||
> How about:
>
> Shares approximate Wi-Fi and cellular location data with &vendorShortName;
> to improve our geolocation service
More concise, which is good for android settings view since the text wraps heavily and occupies a lot of space in the view (and there is a lot of items battling for space in the prefs view already).
Mika is this ok instead?
Flags: needinfo?(udevi)
Comment 33•10 years ago
|
||
(In reply to Garvan Keeley [:garvank] from comment #32) > > How about: > > > > Shares approximate Wi-Fi and cellular location data with &vendorShortName; > > to improve our geolocation service LGTM, though we lose the "of your device" phrase, which might be important for explaining the privacy issue.
Assignee | ||
Comment 34•10 years ago
|
||
And can we go even shorter, removing the words 'approximate' and 'data': Shares Wi-Fi and cellular location with &vendorShortName; to improve our geolocation service
Assignee | ||
Comment 35•10 years ago
|
||
I haven't been able to get ahold of Mika, is there a plan B? I need to get this landed tomorrow. The short and sweet looks great on-screen: https://www.dropbox.com/s/x8f5s6yhydsgj7e/Screenshot_2014-10-09-22-38-16.png?dl=0
Flags: needinfo?(cpeterson)
Comment 36•10 years ago
|
||
LGTM. Let's land this and we can tweak the text later, if needed. One last bikeshed comment: maybe we should include the word "your" to emphasize the privacy aspect. The FHR text in your screenshot uses the word "your". -"Shares Wi-Fi and cellular location with &vendorShortName; to improve our geolocation service" +"Shares your Wi-Fi and cellular location with &vendorShortName; to improve our geolocation service"
Flags: needinfo?(cpeterson)
Comment 37•10 years ago
|
||
This is final. Please use this: "Shares approximate Wi-Fi and cellular location of your device with &vendorShortName; to improve our geolocation service"
Flags: needinfo?(udevi)
Assignee | ||
Comment 38•10 years ago
|
||
This is with Mika's latest wording: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=e62514f06659
Attachment #8500803 -
Attachment is obsolete: true
Attachment #8503381 -
Flags: review?(rnewman)
Updated•10 years ago
|
Attachment #8503381 -
Flags: review?(rnewman) → review+
Keywords: checkin-needed
Comment 39•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/4978e73d4444
Keywords: checkin-needed
Whiteboard: geolocation, mozilla location services → geolocation, mozilla location services[fixed-in-fx-team]
Comment 40•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4978e73d4444
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: geolocation, mozilla location services[fixed-in-fx-team] → geolocation, mozilla location services
Target Milestone: --- → Firefox 35
You need to log in
before you can comment on or make changes to this bug.
Description
•