Last Comment Bug 784580 - Crash when panning/zooming leaflet map site with many markers
: Crash when panning/zooming leaflet map site with many markers
Status: NEW
: crash, reproducible
Product: Firefox for Android
Classification: Client Software
Component: General (show other bugs)
: Trunk
: ARM Android
: -- critical (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
: Sebastian Kaspari (:sebastian)
Depends on:
  Show dependency treegraph
Reported: 2012-08-21 20:32 PDT by William Lachance (:wlach)
Modified: 2013-06-25 08:55 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Logcat just before crash (154.14 KB, text/plain)
2012-08-22 08:42 PDT, William Lachance (:wlach)
no flags Details
Part 1: Define state directory in mach context object (4.52 KB, patch)
2013-06-22 18:42 PDT, Gregory Szorc [:gps]
k0scist: review+
Details | Diff | Splinter Review

Description William Lachance (:wlach) 2012-08-21 20:32:41 PDT
I just knocked up a mobile version of my nixi site ( using leaflet.js and tried loading it with Firefox for Android to see what would happen.

Here's the URL:

(I'll try to keep it up for a while)

Performance in general is janky, but more importantly it's very easy to trigger a crash just by panning/zooming in on the page. Just move to the center and pinch-zoom in and out repeatedly.

This is reproducible on all versions of Firefox for Android I tried (beta, Aurora, and Nightly).
Comment 1 Scoobidiver (away) 2012-08-22 01:46:58 PDT
Can you provide the crash ID from about:crashes?
Comment 2 William Lachance (:wlach) 2012-08-22 07:52:30 PDT
1. I forgot to mention that this is on a Galaxy Nexus running Android 4.1. Haven't tried other platforms yet.

2. Weirdly enough, I don't see these crashes in about:crashes? Maybe it's due to some kind of out-of-memory condition that prevented the crash from being reported.

3. I also just realized that the example I put up wouldn't really work outside of Montreal. I put up a new version here:

4. I also did some digging and found that people have had similar problems with other sites and the android stock browser:

There's a suggestion in the bug to set `window.L_DISABLE_3D = true` before including Leaflet (to disable hardware acceleration). Doing so does indeed seem to fix the crash here.
Comment 3 Scoobidiver (away) 2012-08-22 07:57:05 PDT
At least, do you have a logcat?
Comment 4 William Lachance (:wlach) 2012-08-22 08:42:44 PDT
Created attachment 654228 [details]
Logcat just before crash
Comment 5 William Lachance (:wlach) 2012-08-22 08:43:29 PDT
Attached a logcat, nothing really interesting that I could see. I also ran top to see what was going on:

 8289  1  22% S    40 734792K 203400K  fg u0_a60   org.mozilla.fennec_aurora
 8289  1   1% S    40 726152K 184512K  fg u0_a60   org.mozilla.fennec_aurora
 8289  0  39% S    40 741752K 208488K  fg u0_a60   org.mozilla.fennec_aurora
 8289  0  52% S    40 760212K 235508K  fg u0_a60   org.mozilla.fennec_aurora
 8289  0  52% S    40 720268K 184020K  fg u0_a60   org.mozilla.fennec_aurora
 8289  0  55% S    40 955752K 378488K  fg u0_a60   org.mozilla.fennec_aurora

It looks like RSS shoots right up just before the crash.
Comment 6 Kartikaya Gupta ( 2012-09-25 14:41:09 PDT
If it is caused by the screenshot code (as you surmised in your blog post) then bug 791263 should help here.
Comment 7 Gregory Szorc [:gps] 2013-06-22 18:42:20 PDT
Created attachment 766373 [details] [diff] [review]
Part 1: Define state directory in mach context object

I'm gonna take a quick stab at this. But, it requires a new feature in
the mach core.
Comment 8 Jeff Hammel 2013-06-24 16:23:35 PDT
Comment on attachment 766373 [details] [diff] [review]
Part 1: Define state directory in mach context object

lgtm; some comments might be nice
Comment 9 Gregory Szorc [:gps] 2013-06-24 16:29:39 PDT
I accidentally. Bug 794580.
Comment 10 William Lachance (:wlach) 2013-06-24 16:37:56 PDT
For what it's worth, this site still reliable crashes fennec, and doesn't produce any bug reports.

Note You need to log in before you can comment on or make changes to this bug.