Last Comment Bug 785631 - Temporarily disable permission manager app awareness
: Temporarily disable permission manager app awareness
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: General (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla17
Assigned To: Mounir Lamouri (:mounir)
:
:
Mentors:
Depends on: 777072
Blocks: 785632
  Show dependency treegraph
 
Reported: 2012-08-25 09:40 PDT by Mounir Lamouri (:mounir)
Modified: 2012-08-30 06:41 PDT (History)
3 users (show)
mounir: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---
-


Attachments
Patch (1.50 KB, patch)
2012-08-25 20:08 PDT, Mounir Lamouri (:mounir)
cjones.bugs: review+
Details | Diff | Splinter Review

Description Mounir Lamouri (:mounir) 2012-08-25 09:40:59 PDT
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.
Comment 1 Mounir Lamouri (:mounir) 2012-08-25 20:08:31 PDT
Created attachment 655394 [details] [diff] [review]
Patch
Comment 2 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-08-25 20:10:24 PDT
Comment on attachment 655394 [details] [diff] [review]
Patch

s/find/found/
Comment 3 Ryan VanderMeulen [:RyanVM] 2012-08-26 19:25:52 PDT
https://hg.mozilla.org/mozilla-central/rev/bf3e9e62d5de
Comment 4 Andrew Overholt [:overholt] 2012-08-30 06:41:13 PDT
Temporary fix seems like it's not a blocker (plus it's done).

Note You need to log in before you can comment on or make changes to this bug.