618581 - IndexedDB: Adjust quota for mobile use

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Ben Turner (not reading bugmail, use the needinfo flag!)]

IndexedDB is designed to allow a website to store large quantities of data on disk somewhere. For desktop Firefox we prompt once to allow a website to create a database and then allow up to 50 MB of storage. If the website tries to use more than 50 MB we prompt again and, if the user agrees, we allow unlimited storage. I'm not sure how Fennec will want to handle this, but at the very least I expec that 50 MB is a bit large. That pref lives here:

https://mxr.mozilla.org/mozilla-central/source/modules/libpref/src/init/all.js#102

Comment 1

[Doug Turner (:dougt)]

nothing to do here until indexeddb works in e10s.

Comment 2

[Alon Zakai (:azakai)]

Attachment: patch?

Not sure what we want to set this to. How about 5MB?

Although most mobile phones these days have a few GB of storage, so maybe this isn't that important...

Comment 3

[Shawn Wilsher :sdwilsh]

Is indexedDB even turned on for mobile yet?

Comment 4

[Alon Zakai (:azakai)]

There is a patch for that, waiting for review from bent, bug 619494.

Comment 5

[Doug Turner (:dougt)]

depends where the data is stored.  if the db is on the sdcard, it probably doesn't matter much..  however, if it is on internal memory, them maybe a lower limit is better.

Comment 6

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

Comment on attachment 525090 [details] [diff] [review]
patch?

This is good for now. I was going to suggest using the sdcard, but I realize a problem:
* the sdcard might not exist whn the file is first created. we can't change locations now, the previously stored data would be lost
* the sdcard could be removed, causing all the data to be lost.

I think we should keep the data with the profile. We can already move the profile to an sdcard, so that's what we have to work with. Let's not add any extra complexity.

Comment 7

[Alon Zakai (:azakai)]

http://hg.mozilla.org/mozilla-central/rev/4d304d0b409d

Comment 8

[Anna (Waverley)]

Can someone please provide some steps to reproduce for this bug?

Comment 9

[Alon Zakai (:azakai)]

STR could be to visit a website that uses IndexedDB, and requests 10MB more after the initial granting of permission. With this bug fixed, we should prompt for allowing the additional 10MB.