We need to fix this. Lots of MMS tasks are under way and rely on the Message app working. CC'ing related people. I'm wondering something regarding the contact changes produces the above logcat error. I'm sure it's not due to bug 831683 because it happened before my patches.
Keywords: regression, regressionwindow-wanted
I was working on finding the regression window for this bug using the m-c builds and here is what I found— 1. The last time the Notifications show up, was on the 2013-01-14-03-05-31 build. (The messages in the Inbox were still opening up tho). 2. The last time the messages could be opened was on the 2013-01-24-16-28-01 build.
Could any Gaia member please take a look on this bug? We cannot test the Message App with m-c build, where most of the Gecko engineers are working. I guess there might be some Gecko modifications in the Contact stuff and the Message App doesn't handle the returned "null" properly. b2g-18 branches are not affected.
I think that it's related with some changes in Contacts API. I will come back with more feedback.
I have a look to this and I've seen the problem is regarding permissions. PermissionPromptHelper is not working properly and is denying the access when one of the permission is not requested. For contacts permission can be 'read', 'write' and 'create', as sms is requesting only read permission PermissionPromptHelper is denying the access to contacts. As a workaround modifying permission from readonly to readwrite it will work until bug 804623 is resolved.
Thanks Albert! ;)
Gene, Is SMS failing yet?
The work-around at comment #6 works for me. :) Thanks Albert and Borja!
Hi Gene, The thing is that... it's a workaround! What it means that with the code in 'master' Gaia & m-c Gecko it's not working at all... We need to fix the permission stuff in Gecko. Could you assign this bug to the one in charge of this? Thanks.
(In reply to Borja Salguero [:borjasalguero] from comment #10) > Hi Gene, > The thing is that... it's a workaround! What it means that with the code in > 'master' Gaia & m-c Gecko it's not working at all... We need to fix the > permission stuff in Gecko. Could you assign this bug to the one in charge of > this? Thanks. Wait! Aren't we solving the Gecko part already at bug 804623? It seems Gregor is looking at that. For this bug here, we can apply the workaround on the Gaia end first. Does that sound reasonable to you? Hi Steve, would you mind taking this task for the Gaia part?
Component: General → Gaia::SMS
It's just a small change in manifest. Borja, do you agree to apply this workaround in master for MMS developing?
I Steve. Despite of we could workaround this issue, I would like to wait until getting feedback from Gecko (due to they have detect where the bug is).
(In reply to Borja Salguero [:borjasalguero] from comment #13) > I Steve. Despite of we could workaround this issue, I would like to wait > until getting feedback from Gecko (due to they have detect where the bug is). Bug 804623 landed. It should be fixed now if you update gecko and gaia.
Given the STR, this issue no longer exists
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.