Closed Bug 1434469 Opened 4 years ago Closed 4 years ago

Add a permission for coarse location data

Categories

(Firefox for Android Graveyard :: General, enhancement, P2)

All
Android
enhancement

Tracking

(fennec+, firefox58 wontfix, firefox59 wontfix, firefox60 fixed)

RESOLVED FIXED
Firefox 60
Tracking Status
fennec + ---
firefox58 --- wontfix
firefox59 --- wontfix
firefox60 --- fixed

People

(Reporter: mkaply, Assigned: jchen)

Details

Attachments

(1 file)

For some reason, the Google CTS doesn't like you requesting FINE without requesting COARSE. 

And they really don't like you requesting FINE twice (which I think we don't do anymore).

We should add the COARSE permission. It won't affect anything and will allow us to better pass the CTS.
tracking-fennec: --- → ?
OS: Unspecified → Android
Hardware: Unspecified → All
Version: unspecified → Trunk
Assignee: nobody → nchen
tracking-fennec: ? → +
Priority: -- → P2
To be clear, I think this bug is about updating the AndroidManifest so it declares both coarse and fine location access.

[triage] iiuc, seems non-critical but if Jim is going to fix it, wfm. Please mark P3 if you decide not to though.
Yes, that is correct. We should declare both (and make sure we don't declare FINE twice which I think was fixed when we removed the search widget).
Comment on attachment 8950659 [details]
Bug 1434469 - Add ACCESS_COARSE_LOCATION to list of permissions;

https://reviewboard.mozilla.org/r/219906/#review225744
Attachment #8950659 - Flags: review?(snorp) → review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/12e36be8f260
Add ACCESS_COARSE_LOCATION to list of permissions; r=snorp
https://hg.mozilla.org/mozilla-central/rev/12e36be8f260
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.