821998 - Installed web apps get stuck on splashscreen - do not finish loading

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Web Apps (PWAs), defect)

Description

[Matěj Cepl]

Attachment: about:support data

I have number of apps installed from the Marketplace and suddenly with the current Nightly on Android (see about:support data attached) they don't start at all (tried Hesla, Twitter, Kohoutkova, openMarkers). All of them show the startup screen and that's it. The actual application doesn't show up.

Comment 1

[Matěj Cepl]

Attachment: relevant part of logcat

Hmm, what about

E/GeckoConsole(18773): [JavaScript Warning: "test for equality (==) mistyped as assignment (=)?" {file: "jar:jar:file:///data/app/org.mozilla.fennec-2.apk!/omni.ja!/components/nsDefaultCLH.js" line: 47 column: 60 source: "tDir = cmdLine.handleFlagWithParam("print-xpcom-dir", false)) {
E/GeckoConsole(18773): "}]
E/GeckoConsole(18773): [JavaScript Warning: "test for equality (==) mistyped as assignment (=)?" {file: "jar:jar:file:///data/app/org.mozilla.fennec-2.apk!/omni.ja!/components/nsDefaultCLH.js" line: 62 column: 60 source: "                                                      false)) {
E/GeckoConsole(18773): "}]

Is really Gecko apps engine so delicate that it cannot handle some JavaScript typos? That's a bad news.

Comment 2

[Jason Smith [:jsmith]]

Is this bug reproducible after doing this multiple times?

Can you give exact steps to reproduce?

Comment 3

[Wesley Johnston (:wesj)]

I've been seeing this too. I think something changed and we're not getting the right state start/stop messages (i.e. we never show the throbber either).

Comment 4

[Wesley Johnston (:wesj)]

More relevant log line is probably:

E/GeckoConsole(18773): [JavaScript Error: "ReferenceError: user_pref is not defined" {file: "chrome://browser/content/WebAppRT.js" line: 30}]

which is fallout from bug 737596.

Comment 5

[Théo Chevalier]

Attachment: Patch

Ho, I'm really sorry guys :(

Can you please try again with this build? https://tbpl.mozilla.org/?tree=Try&rev=1de864f5e8da

If it's still not working, a quick temp fix could be:

pref("toolkit.telemetry.prompted", 2),
pref("toolkit.telemetry.notifiedOptOut", 2)

Comment 6

[Théo Chevalier]

Confirmed, my Twitter webapp is working again with this patch

Comment 7

[Matěj Cepl]

(In reply to Théo Chevalier [:tchevalier] from comment #5)
> If it's still not working, a quick temp fix could be:
> 
> pref("toolkit.telemetry.prompted", 2),
> pref("toolkit.telemetry.notifiedOptOut", 2)

How can I set it up on my phone (sorry, I don't build my own FF)?

Comment 8

[Théo Chevalier]

(In reply to Matej Cepl from comment #7)
> (In reply to Théo Chevalier [:tchevalier] from comment #5)
> > If it's still not working, a quick temp fix could be:
> > 
> > pref("toolkit.telemetry.prompted", 2),
> > pref("toolkit.telemetry.notifiedOptOut", 2)
> 
> How can I set it up on my phone (sorry, I don't build my own FF)?

You don't have to set up anything, just to download & install a try build here if you want to confirm: http://ftp-scl3.mozilla.com/pub/mozilla.org/firefox/try-builds/[e-mail]-1de864f5e8da/
(It worked for me)

Tommorrow's Nightly should be fine. Thanks for pointing this out, btw.

Pushed to inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c8a1314aa449

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/c8a1314aa449

Comment 10

[Aaron Train [:aaronmt]]

Verified fixed on mozilla-inbound

Comment 11

[Matěj Cepl]

Confirming, the binary from http://ftp-scl3.mozilla.com/pub/mozilla.org/firefox/try-builds/[e-mail]-1de864f5e8da/try-android/fennec-20.0a1.en-US.android-arm.apk works for me as well. Thank you!