'geolocation' permission should be set to prompt for certified apps

VERIFIED FIXED in Firefox 21

Status

()

Core
Permission Manager
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: gwagner, Assigned: gwagner)

Tracking

(Blocks: 1 bug, {dev-doc-needed})

unspecified
mozilla21
x86
Mac OS X
dev-doc-needed
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(blocking-basecamp:+, firefox19 wontfix, firefox20 wontfix, firefox21 fixed, b2g18 fixed)

Details

(Whiteboard: [needs-perms-matrix-update])

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 699173 [details] [diff] [review]
patch
Assignee: nobody → anygregor
Attachment #699173 - Flags: review?(jonas)
(Assignee)

Comment 2

6 years ago
I will let Jonas do the reasoning :) 
Basically, E.me runs in the homescreen and since the homescreen is a certified app, E.me doesn't prompt for geolocation.
blocking-basecamp: --- → ?

Updated

6 years ago
Keywords: dev-doc-needed
Whiteboard: [needs-perms-matrix-update]

Updated

6 years ago
Blocks: 783741
Makes sense, but I have a question (that I brought up in person):

If we now make geolocation PROMPT_ACTION for certified apps, shouldn't we know see those certified apps in the settings apps perms UI as a result and be able to control the geolocation permissions off that UI?
Comment on attachment 699173 [details] [diff] [review]
patch

Review of attachment 699173 [details] [diff] [review]:
-----------------------------------------------------------------

Let's do this for now, but we should bring this up on the mailing list.
Attachment #699173 - Flags: review?(jonas) → review+
Putting needsinfo on Josh to think about the UX implications here.

comment 3 would be an example to think about.
Flags: needinfo?(jcarpenter)
(Assignee)

Updated

6 years ago
Blocks: 823974
Talked with jcarpenter in person - we're just going to reuse the existing apps perms UI for v1.
Flags: needinfo?(jcarpenter)

Updated

6 years ago
blocking-basecamp: ? → +
https://hg.mozilla.org/mozilla-central/rev/d13052681690
https://hg.mozilla.org/mozilla-central/rev/0f28e7c859c8
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
https://hg.mozilla.org/releases/mozilla-b2g18/rev/2fb4aa15cc2f
https://hg.mozilla.org/releases/mozilla-b2g18/rev/9335c82bcbc5
status-b2g18: --- → fixed
status-firefox19: --- → wontfix
status-firefox20: --- → wontfix
status-firefox21: --- → fixed

Updated

6 years ago
Keywords: verifyme
QA Contact: jsmith
Confirmed we are getting prompts, but the remember my choice default isn't right. Followup coming.
Status: RESOLVED → VERIFIED
Keywords: verifyme

Updated

6 years ago
Depends on: 828209

Updated

6 years ago
Depends on: 828215

Updated

6 years ago
No longer depends on: 828215
You need to log in before you can comment on or make changes to this bug.