644790 - Fennec crash on startup [@ nsCycleCollector::BeginCollection]

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Kevin Brosnan [Ex-Mozilla]]

Attachment: crash log

Updated to today's build. Now is crashing on startup. See attached log. Had a crash report submitted but I can't reach it by about:crashes. Where is the profile folder in the android file system?

Comment 1

[Dave Miller [:justdave]]

I'm crashing on startup on HTC Evo 4G running Android 2.2 also.  Likewise, I can't get in far enough to run about:crashes, and the debugger's telling me org.mozilla.fennec doesn't exist when I try to look on the filesystem.

Comment 2

[Aakash Desai [:aakashd]]

Is this on the RC as well? If so, we should add this to the unsupported devices list.

Comment 3

[Kevin Brosnan [Ex-Mozilla]]

4.0rc1 starts, bit sluggish but that is the phone.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I bisected tinderbox builds on my Archos A32, which led to the regression range being:

http://ftp.mozilla.org/pub/mozilla.org/mobile/tinderbox-builds/mozilla-central-android-r7/1300976095/
http://hg.mozilla.org/mozilla-central/rev/24714b2ebd4f
starts fine

http://ftp.mozilla.org/pub/mozilla.org/mobile/tinderbox-builds/mozilla-central-android-r7/1300977747/
http://hg.mozilla.org/mozilla-central/rev/b97a060746f9
crashes

That means the regression was in this push:
http://hg.mozilla.org/mozilla-central/pushloghtml?changeset=b97a060746f9
(unless there were mobile-browser changes in that window... but that seems unlikely, though I haven't checked)

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Two crash reports submitted using:
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/2011/03/2011-03-24-12-mozilla-central-android-r7/
and then I downgraded to
http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/2011/03/2011-03-24-04-mozilla-central-android-r7/
in order to get to about:crashes are:
bp-d9137e70-c042-49c3-8ce0-de6bd2110324
bp-5c49ddae-7cf2-40e8-b58d-ac1842110324

Comment 6

[Michael Wu [:mwu]]

Please back out if it can be done cleanly.

There are two issues with the patches in bug 620931

1. The android code uses -omnijar and bug 620931 does not fix
GeckoAppShell.java which uses it.
2. The code adds an additional assumption that the omnijar is named omni.jar.
This is not at all the case. This assumption should be removed since the
omnijar is the apk on android, and in the future, we could even make the
omnijar the firefox executable itself.

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I backed out the entirety of the push that caused this.  The backout is:
http://hg.mozilla.org/mozilla-central/pushloghtml?changeset=e11c2f95f781

Comment 8

[Daniel Holbert [:dholbert]]

This seems to be fixed in today's build, so it looks like the backout was effective.

However, any end-users who (automatically) got yesterday's build now have a non-functional Fennec installation on their device.  It can't start up, so it can't update itself.  That's a pretty bad spot to be in.

Given that it took some bisecting to find out when this broke, I'm assuming this bug didn't turn any tinderboxen orange.  Do we need more automated android testing  infrastructure to catch these sorts of things (startup-crashes blocking any activity) in the future?

Comment 9

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #8)

> Given that it took some bisecting to find out when this broke, I'm assuming
> this bug didn't turn any tinderboxen orange.  Do we need more automated android
> testing  infrastructure to catch these sorts of things (startup-crashes
> blocking any activity) in the future?

The existing tinderbox automation should have caught the problem, but it was purple from unrelated issues.

Comment 10

[Mike Hommey [:glandium]]

It would help, though, to have these tests run on try, too. I would have caught the problem before landing. I could have tried the try build on an android device, but at the time I was baking the patch queue, I didn't have one.

Anyways, sorry for the mess, and sorry not to have noticed earlier. It also doesn't help that mobile tbpl is separated.

Comment 12

[Brad Lassey [:blassey] (use needinfo?)]

fixed by backout:
http://hg.mozilla.org/mozilla-central/rev/1492b6e75639
http://hg.mozilla.org/mozilla-central/rev/941d126f6dd2
http://hg.mozilla.org/mozilla-central/rev/08d1aeeea824
http://hg.mozilla.org/mozilla-central/rev/16e48d6b3b9c
http://hg.mozilla.org/mozilla-central/rev/5deb267b1d33

Comment 13

[Cristian Nicolae (:xti)]

Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:8.0a1)Gecko/20110905
Firefox/8.0a1 Fennec/8.0a1
Device: Samsung Galaxy S
OS: Android 2.2