Closed
Bug 827806
Opened 12 years ago
Closed 12 years ago
'geolocation' permission should be set to prompt for certified apps
Categories
(Core :: Permission Manager, defect)
Tracking
()
People
(Reporter: gwagner, Assigned: gwagner)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-needed, Whiteboard: [needs-perms-matrix-update])
Attachments
(1 file)
(deleted),
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
Assignee: nobody → anygregor
Attachment #699173 -
Flags: review?(jonas)
Assignee | ||
Comment 2•12 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•12 years ago
|
Keywords: dev-doc-needed
Whiteboard: [needs-perms-matrix-update]
Updated•12 years ago
|
Blocks: Apps-Dev-Doc-Needed
Comment 3•12 years ago
|
||
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+
Comment 5•12 years ago
|
||
Putting needsinfo on Josh to think about the UX implications here.
comment 3 would be an example to think about.
Flags: needinfo?(jcarpenter)
Assignee | ||
Comment 6•12 years ago
|
||
Comment 7•12 years ago
|
||
Talked with jcarpenter in person - we're just going to reuse the existing apps perms UI for v1.
Flags: needinfo?(jcarpenter)
Assignee | ||
Comment 8•12 years ago
|
||
Updated•12 years ago
|
blocking-basecamp: ? → +
Comment 9•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d13052681690
https://hg.mozilla.org/mozilla-central/rev/0f28e7c859c8
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment 10•12 years ago
|
||
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
Comment 11•12 years ago
|
||
Confirmed we are getting prompts, but the remember my choice default isn't right. Followup coming.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•