Closed Bug 1095451 Opened 7 years ago Closed 7 years ago

Gaia not loading on current Mulet

Categories

(Firefox OS Graveyard :: Runtime, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2.1 S9 (21Nov)

People

(Reporter: gerard-majax, Assigned: gerard-majax)

References

Details

(Keywords: regression, Whiteboard: [systemsfe])

Attachments

(1 file, 1 obsolete file)

On current gecko/gaia master, mulet loads with an empty tab:
 - unable to change the URL
 - it's blocked on about:start
 - no content loaded
 - b2g shell.html not loaded as it should have been
Reverting the pref changes from bug 1093691, my Mulet starts again.
Depends on: 1093691
Keywords: regression
Attachment #8518886 - Flags: review?(gavin.sharp)
Comment on attachment 8518886 [details] [diff] [review]
Switch off e10s for Mulet

Review of attachment 8518886 [details] [diff] [review]:
-----------------------------------------------------------------

I think it's better in firefox.js if you add a && !defined (MOZ_MULET) to where we're setting that pref to true
(Unless the possibility of running Mulet using an e10s'fied version of Firefox is not in a distant future)
(In reply to :Felipe Gomes from comment #5)
> (Unless the possibility of running Mulet using an e10s'fied version of
> Firefox is not in a distant future)

I have no idea on this, but I don't have any problem fixing it in firefox.js directly.
Attached patch Switch off e10s for Mulet (obsolete) — Splinter Review
Attachment #8518886 - Attachment is obsolete: true
Attachment #8518886 - Flags: review?(gavin.sharp)
Attachment #8519015 - Flags: review?(gavin.sharp)
Comment on attachment 8518886 [details] [diff] [review]
Switch off e10s for Mulet

I think I prefer this patch, actually - removes the mulet stuff from firefox proper.
Attachment #8518886 - Flags: feedback+
Attachment #8519015 - Flags: review?(gavin.sharp) → review-
Attachment #8518886 - Attachment is obsolete: false
Whiteboard: [systemsfe]
Attachment #8519015 - Attachment is obsolete: true
Comment on attachment 8518886 [details] [diff] [review]
Switch off e10s for Mulet

Gavin, since you prefer this one :)
Attachment #8518886 - Flags: review?(gavin.sharp)
Attachment #8518886 - Flags: review?(gavin.sharp) → review+
(In reply to Alexandre LISSY :gerard-majax from comment #10)
> Pending try: https://tbpl.mozilla.org/?tree=Try&rev=4c67fd56445a

Green.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/fac2cfed1419
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S9 (21Nov)
You need to log in before you can comment on or make changes to this bug.