896958 - [MMS] Showing wrong error message when opening the SMS application after the home button is pressed

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Leo]

1. Title: Showing wrong error message when opening the SMS application after the home button is pressed
2. Precondition: SMS app should be working
3. Tester's Action:  (1) Receive MMS with image or audio or video file
                     (2) Click on any received image or music file or video file
                     (3) Press "Home" button
                     (4) Open SMS application 
4. Detailed Symptom (ENG.) : 'There is no application available to open this file' error window is shown
5. Expected:  When message application is open after clicking on the Home button,this error window should not be shown
6. Reproducibility: Y
1) Frequency Rate : 100%
7. Gaia Master/v1-train: Reproduced on v1-train
8. Gaia Revision:  bbad03e30527e1df8261a462e5cefbdc562f0819
9. Personal email id: [e-mail]

Comment 1

[Leo]

Attachment: pointer to pull request

Please review the patch and provide your comments

Comment 2

[Borja Salguero [:borjasalguero]]

Small comment in your PR, please address the change and I'll test this again! Thanks

Comment 3

[Leo]

Hi Borja,
I have added the comments in PR.Can you please check that and update.
Thanks

Comment 4

[Borja Salguero [:borjasalguero]]

Hi Leo, please re-check the PR :)

Comment 5

[Leo]

(In reply to Borja Salguero [:borjasalguero] from comment #4)
> Hi Leo, please re-check the PR :)

Hi Borja,

Tested and updated the PR as per the comments.
Please check.

Thanks,

Comment 6

[Borja Salguero [:borjasalguero]]

Comment on attachment 780192 [details]
pointer to pull request

Probably we could add this to leo+, due to is a surgical fix. Wdyt?

Comment 7

[Jinyoon]

I set leo+ because this is vendor's QE issue and hope to landed in V1-train for sync-up with vendor build.

Comment 8

[Borja Salguero [:borjasalguero]]

Leo, Why is this not landed yet? Can I help you with this? It's r+ and leo+. Let me know!

Comment 9

[Leo]

Hi Borja,
We dont have permissions for uplifting.
So,Please land this issue on behalf of Leo.
Thanks,

Comment 10

[Borja Salguero [:borjasalguero]]

https://github.com/mozilla-b2g/gaia/commit/fc22e7385dccf60ada2e48f17c465d8662317e44
https://github.com/leob2g/gaia/commit/b57cc2c66964c5d1829e90a7a9d75274dcfe1051

John, could you take a look to this uplift? Thanks!

Comment 11

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted fc22e7385dccf60ada2e48f17c465d8662317e44 to:
v1-train: caa91ce227fe3749fe669761db89706a1f2de959

Comment 12

[Tim Guan-tin Chien [:timdream] (please needinfo)]

v1.1.0hd: caa91ce227fe3749fe669761db89706a1f2de959