Closed Bug 758357 Opened 13 years ago Closed 12 years ago

Only show prompt after 50mb of quota was reached

Categories

(Core :: Storage: IndexedDB, defect)

Product:
Core
Component:
Storage: IndexedDB
Platform:
x86_64
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla16

People

(Reporter: taras.mozilla, Assigned: bent.mozilla)

References

(Depends on 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

Patch
(deleted), patch
sicking
: review+
khuey
: review-
Details | Diff | Splinter Review
patch
(deleted), patch
fabrice
: review+
Details | Diff | Splinter Review
bug 742822 will make indexeddb less persistent. We should proactively get rid of the prompt so web developers can start using indexeddb(and drop dom storage) while we wrap up the quota code.
We should definitely keep this option in mind in order to enable unprefixing IndexedDB. Ideally we would implement the new storage policy of not storing things permanently, but if this is the only thing standing between us and unprefixing, then this seems like a viable short-term fix.
Jonas, should I fix this bug first ?
Jonas agreed on IRC
Assignee: nobody → Jan.Varga
Status: NEW → ASSIGNED
Attached patch Patch (obsolete) (deleted) — — Splinter Review
This keeps it at 50mb like we agreed, and it fixes the UI to be sane (it does nearly the same thing as the fullscreen permission UI). Seems like a fair compromise.
Assignee: Jan.Varga → bent.mozilla
Attachment #638104 - Flags: review?(jonas)
Attached patch Patch (deleted) — — Splinter Review
Clearly that was not the right patch.
Attachment #638104 - Attachment is obsolete: true
Attachment #638104 - Flags: review?(jonas)
Attachment #638105 - Flags: review?(jonas)
Summary: Only show prompt after 100mb of quota was reached → Only show prompt after 50mb of quota was reached
Attachment #638105 - Attachment is patch: true
Comment on attachment 638105 [details] [diff] [review] Patch Review of attachment 638105 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/indexedDB/test/helpers.js @@ +64,1 @@ > }); This was commented out for a reason. It's causing massive orange on tinderbox.
Attachment #638105 - Flags: review-
Jonas checked in a follow-up patch to avoid oranges, and it broke things again. (Irony++) https://hg.mozilla.org/integration/mozilla-inbound/rev/a8bfe5a447e7 Follow-up backed out: https://hg.mozilla.org/integration/mozilla-inbound/rev/3b30ca51a2a1
Looks like a8bfe5a447e7 was almost correct, you just need to make sure you call SimpleTest.finish().
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #7) > This was commented out for a reason. It's causing massive orange on > tinderbox. Somehow I missed that change. Did you look into the reason this causes problems?
OK, I've pushed a8bfe5a447e7 with SimpleTest.finish() added per comment 10. If this doesn't clear things up, I'm going to back out the entire set of patches for this bug. https://hg.mozilla.org/integration/mozilla-inbound/rev/540cd3aabdd4
(In reply to Ryan VanderMeulen from comment #12) > OK, I've pushed a8bfe5a447e7 with SimpleTest.finish() added per comment 10. > If this doesn't clear things up, I'm going to back out the entire set of > patches for this bug. > https://hg.mozilla.org/integration/mozilla-inbound/rev/540cd3aabdd4 Looking much better on TBPL. https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=540cd3aabdd4
(In reply to ben turner [:bent] from comment #11) > (In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #7) > > This was commented out for a reason. It's causing massive orange on > > tinderbox. > > Somehow I missed that change. Did you look into the reason this causes > problems? I haven't had time to investigate it yet.
Sigh. B2G will need to stop setting the "indexedDB" permission for gaia apps. It will still need to set the "indexedDB-unlimited" permission.
(In reply to ben turner [:bent] from comment #17) > Sigh. B2G will need to stop setting the "indexedDB" permission for gaia > apps. It will still need to set the "indexedDB-unlimited" permission. So the solution is to synchronize this landing with removing the indexedDB permission in gaia?
Attached patch patch (deleted) — — Splinter Review
Remove indexedDB permission for B2G
Yes, flipping this in Gaia and Gecko at the same time should fix things.
Attachment #638477 - Flags: review?(fabrice)
Attachment #638477 - Flags: review?(fabrice) → review+
(In reply to Jonas Sicking (:sicking) from comment #21) > Yes, flipping this in Gaia and Gecko at the same time should fix things. And we are in a situation again where we will break indexeddb because we can't land this patch at the same time in Gaia and Gecko. People are using hg-inbound, hg-mc and the github-mc gecko tree.... So the whole patch here is backed out right?
(In reply to Gregor Wagner [:gwagner] from comment #22) > (In reply to Jonas Sicking (:sicking) from comment #21) > > Yes, flipping this in Gaia and Gecko at the same time should fix things. > > And we are in a situation again where we will break indexeddb because we > can't land this patch at the same time in Gaia and Gecko. People are using > hg-inbound, hg-mc and the github-mc gecko tree.... > Ah it shouldn't be a problem because the permission is set in gecko. It's too early for me.
https://hg.mozilla.org/mozilla-central/rev/e7ccacd50d63 https://hg.mozilla.org/mozilla-central/rev/0504d9475cd5
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Depends on: 786769
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: