1027555 - [Vertical Homescreen] No icons on homescreen with DEBUG=1 mode on nightly

Status: RESOLVED FIXED

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

Description

[Sherman Chen [:chens] (inactive)]

Attachment: Homescreen-no-icon.png

Steps to Reproduce:

1. Build Gaia with DEBUG=1 flag
2. Running Gaia on Nightly

Actual Results:
No icons and app name on homescreen.
	 
Expected Results:
Vertical homescreen with icons and app name

Environment Informations:
- Git commit info: 0517d93acd79a070b11f3e3d576fcc34ef4abdb5
- Platform version: 33.0a1(2014-06-18)/32.0a2(2014-06-18)

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Add interapp and system message glue components.

I thought it was failing because of the assertions
we can see on startup about Interapp communication xpcom.
But it still fails with this patch... so I'm not sure it is related.

Comment 2

[Jason Smith [:jsmith]]

Kevin - A bug like this is a dev blocker, right (i.e. block a developer's daily workflow from doing their job)?

Comment 3

[Kevin Grandon :kgrandon]

Yup, it's pretty painful and we should definitely try to fix it if possible.

Comment 4

[Jason Smith [:jsmith]]

Ok - I'll mark this as a 2.0? nom for being a dev workflow blocker.

Comment 5

[Kevin Grandon :kgrandon]

Can we bundle SystemAppProxy.jsm and other necessary components into non-production firefox builds? We did that in the past and that seemed to work well.

Comment 6

[Alexandre Poirot [:ochameau]]

We already have a SystamAppProxy thing in browser helper.

Comment 7

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: race.homescreen.on.nightly.patch

It's a fun race. On desktop the gaia-grid component load and is ready before the app specific code has a change to add an event listener to initialize the rendering.

Shuffling files around fix it for me. I also need this patch pretty badly since I'm in an bug squashing event and this is the core dev env I want to promote. Due to the nature of the patch, r=me, and I will land once the try is green.

Comment 8

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Gaia-Try is green: https://tbpl.mozilla.org/?tree=Gaia-Try&rev=3aed7671c2e1

Let's land it and save the event.

Comment 9

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

https://github.com/mozilla-b2g/gaia/commit/9d2e2b777c6f13a70f4ef3f6c5c759ad4531d01a

Comment 10

[Kevin Grandon :kgrandon]

Comment on attachment 8443882 [details] [diff] [review]
race.homescreen.on.nightly.patch

This is a relatively low risk patch that is needed for the vertical homescreen.

Comment 11

[Kevin Grandon :kgrandon]

2.0: https://github.com/mozilla-b2g/gaia/commit/1d9461622679c9f9624c0e7a4628697659f277cc