Closed Bug 785631 Opened 8 years ago Closed 8 years ago

Temporarily disable permission manager app awareness

Categories

(Core :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17
blocking-basecamp -

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(1 file)

Gaia is generating permissions using the permission manager API which makes any change to the permission manager complex because it will create a window of breakage. The solution is to update the permission manager so the database contains the app id and give time to gaia to use that then, in a second time, update the permission manager so checks for permissions will use the app id.
Doing so will require re-generating a Gaia profile but it will be less annoying because the re-generation has to happen between the two commits and everything will be working in the meantime.
Blocks: 785632
Attached patch PatchSplinter Review
Attachment #655394 - Flags: review?(jones.chris.g)
Comment on attachment 655394 [details] [diff] [review]
Patch

s/find/found/
Attachment #655394 - Flags: review?(jones.chris.g) → review+
Flags: in-testsuite-
Target Milestone: --- → mozilla17
https://hg.mozilla.org/mozilla-central/rev/bf3e9e62d5de
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Temporary fix seems like it's not a blocker (plus it's done).
blocking-basecamp: ? → -
You need to log in before you can comment on or make changes to this bug.