1083198 - [B2G][SMS] Remove fallback for old gecko version in MessageManager.getMessages

Status: RESOLVED FIXED

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

Description

[Augustin Trancart [:autra]]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:32.0) Gecko/20100101 Firefox/32.0
Build ID: 20140924083558

Steps to reproduce:

In message-manager.js:getMessages, there's a fallback to deprecated MozSmsFilter for phone with an old version of gecko.





Expected results:

One day, we should remove it.

Comment 1

[Augustin Trancart [:autra]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25183

Not to be merged now, but when old gecko will reach end-of-life.

Comment 2

[Julien Wajsberg [:julienw] (PTO -> Jan 5)]

end-of-life basically means "when Augustin gets a new phone" :)

Comment 3

[Julien Wajsberg [:julienw] (PTO -> Jan 5)]

Comment on attachment 8505509 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25183

r=me, but please rebase to the new master and push to your PR, so that we can have a proper Gij test run in the gaia-try results.

If everything looks fine (green) then add "checkin-needed" to the "keywords" line on the bug (near the top)

Thanks a lot to help cleaning up all this :)

Comment 4

[Augustin Trancart [:autra]]

Ok things are good except one integration test : https://treeherder.mozilla.org/ui/#/jobs?repo=gaia-try&revision=b74f9f01fbc8

Should I worry about it?

Comment 5

[Julien Wajsberg [:julienw] (PTO -> Jan 5)]

I don't think it's related but I just restarted the test to be on the safe side.

Comment 6

[Carsten Book [:Tomcat]]

https://github.com/mozilla-b2g/gaia/commit/2a67c5fe7b9552490213a48c0310406b28a6d255