891042 - Gaia homescreen is not loading in firefox nightly

Status: RESOLVED FIXED

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

Description

[Kevin Grandon :kgrandon]

Appears a recent gaia commit has broken the homescreen from loading within gaia.

Bisecting I found this as the breaking commit:

01649f946f087dcc1fa791659ad7eb50db4d9cb3 is the first bad commit
commit 01649f946f087dcc1fa791659ad7eb50db4d9cb3
Author: Edgar Chen <[e-mail]>
Date:   Sun Jun 30 10:38:22 2013 +0800

    Bug 874769 - Use IccHelper to access card state. r=salva,etienne,arcturus,alive,arthurcc

:040000 040000 8cf37180b49cfb10186ef5698092df2820ae2a8f 83caa3b2b10700b2df6f293feeabcbcf9a945137 M	apps
:040000 040000 8d161bd64531548eb7100699561e479b23bd919f 26605dac1456e2dbd56b9b7c1df2b18521844afc M	shared

Comment 1

[Kevin Grandon :kgrandon]

Attachment: Github pull request pointer

Comment 2

[Kevin Grandon :kgrandon]

Comment on attachment 772232 [details]
Github pull request pointer

Etienne, Edgar - could one of you briefly review this simple change? Thanks!

Comment 3

[Edgar Chen [:edgar]]

Comment on attachment 772232 [details]
Github pull request pointer

Arthur is more suitable than me to review this. :)

Comment 4

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 772232 [details]
Github pull request pointer

Kevin, thanks for the patch. Please check my comment in github.

Comment 5

[Kevin Grandon :kgrandon]

Arthur - Responded to github, let me know if you have any other concerns. We really need to land something for this asap as it's breaking workflow for a lot of developers.

I think we can land the simple fix, then investigate using a more full-fledged mock for a followup.

Comment 6

[Arthur Chen [:arthurcc][inactive after 6/30]]

Let's do the simple fix. However, please make sure `IccHelpler.enabled` still works. It should return false if mozIccManager is undefined.

Comment 7

[Edgar Chen [:edgar]]

I am not sure is it a mockMobileConnection for desktop in tools/extensions/desktop-helper/content/data/lib/mobile_connection.js [1]. If so, maybe we could add a mockIccManager there.

[1] https://github.com/mozilla-b2g/gaia/blob/master/tools/extensions/desktop-helper/content/data/lib/mobile_connection.js

Comment 8

[Edgar Chen [:edgar]]

Attachment: Mock mozIccManager for desktop, v1

:kgrandon, could you help to try does this patch fix the issue? Thank in advance.

Comment 9

[Arthur Chen [:arthurcc][inactive after 6/30]]

Attachment: link to https://github.com/mozilla-b2g/gaia/pull/10857

Edgar, thanks for the information!
Kevin, I tried to mock mozIccManager as expected. Then we can avoid the hack in IccHelper. Any comments?

Comment 10

[Kevin Grandon :kgrandon]

Comment on attachment 772559 [details]
link to https://github.com/mozilla-b2g/gaia/pull/10857

This looks great! Thank you for knocking this out.

Comment 11

[Kevin Grandon :kgrandon]

This has landed in master: https://github.com/mozilla-b2g/gaia/commit/82e6fd4b52223a8bcec0c78fc0b4aa4d0b1be0e0