522514 - Make tile capacity a preference

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Benjamin Stover (:stechz)]

So that devices with more memory can have a larger tile cache.

Comment 1

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

Attachment: patch

defaults to 25 on maemo & desktop and 15 on wince

Comment 2

[Doug Turner (:dougt)]

tile.cache.capacity


Also add a note that changing this significantly changes memory requirements.  Might also be a good idea to also pref the size of the tiles.

Comment 3

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

(In reply to comment #2)
> tile.cache.capacity

I use "size" in the JS part of the patch

> Also add a note that changing this significantly changes memory requirements. 
> Might also be a good idea to also pref the size of the tiles.

Maybe, but I'll let happen in a separate bug

Comment 4

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

pushed:
https://hg.mozilla.org/mobile-browser/rev/ad0ae2b16416

Comment 5

[Aakash Desai [:aakashd]]

This patch makes Fennec on the n810 really crash for panning heavy sites while they load (i.e. http://en.wikipedia.org/wiki/Michael_Jackson )

build:

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20091020
Fennec/1.0b5pre


...meanwhile this works fine on the n900:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091020
Fennec/1.0b5pre

Comment 6

[Benjamin Stover (:stechz)]

Bug 523396 has been filed to tune n810 capacity down.

Comment 7

[Aakash Desai [:aakashd]]

Thanks for filing the bug Ben, I'm verifying this on builds (omnia 2 and n900):

Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.2b1pre) Gecko/20091020 Fennec/1.0a4pre

and

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091020
Fennec/1.0b5pre