552640 - Fennec hangs on N900 when loading a huge background png image on www.tjareborg.fi

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Pekka Vanhoja]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.6) Gecko/2009011912 Firefox/3.0.6
Build Identifier: Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv1.9.2.1) Gecko/20100126 Firefox/3.6.1 Fennec/1.0.0

When loading the page www.tjareborg.fi the mobile Firefox on N900 hangs the device.

The page has this big image:

http://www.tjareborg.fi/StaticContent/Shared/components/box/img/box-top.png?v=201002093
Dimensions: 3,200px × 10,000px

Reported by Imagemagick 'identify':
$ identify box-top.png 
box-top.png PNG 3200x10000 3200x10000+0+0 DirectClass 8-bit 43.1504kb 0.650u
0:02

The hang looks like to be caused by the heavy swapping caused by handling that image.


Reproducible: Always

Steps to Reproduce:
1. Open www.tjareborg.fi
Actual Results:  
Fennec app is stalled, the whole device is really sluggish.

Expected Results:  
The page is loaded.

Comment 1

[Pekka Vanhoja]

Attachment: Patch to limit image pix dimensions on Maemo builds

This issue was visible also in the microb-engine on N900, this patch was devised to limit the pixel size of images to avoid the hanging situation. The patch introduces a new preference for run-time controlling of the limit.

Comment 2

[Pekka Vanhoja]

This is how I intended the preference to be:

/* Used: PR_GetPhysicalMemorySize / n = delimiter for image dimension in pixels
  -1: default (n = 8, e.g. 256 MB / 8 = 32 MPix)
   0: not limited
   n: the nominator for delimiter calculation
*/
pref("image.dimension.delimiter", -1);

Comment 3

[Pekka Vanhoja]

Ups, restoring the dependencies.

Comment 4

[Martijn Wargers (dead)]

Comment on attachment 432772 [details] [diff] [review]
Patch to limit image pix dimensions on Maemo builds

Stuart, could you perhaps review this patch?

Comment 5

[BMO Automation]

Closing all opened bug in a graveyard component