Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-17 Thread Jonas Sicking
Hi Kinuko and Eric, Thanks for providing feedback from a Google perspective! Definitely still feels like there's a lot of possibilities and options here. No combination of solutions seem obviously best to me, so please do take the below as me thinking out loud rather than having strong opinions.

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-17 Thread Kyaw Tun
Option C) using numeric priority are good enough for most web application. It has advantage of easy to implement. Option C) can be combined with option B) by defining that 0 means temporary, 1 means persistent and undefined means default. Any other values should throw error. Since we can query

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-13 Thread Eric U
Good writeup, Jonas--I think you've hit the major points. I think numeric priorities are both overkill and underpowered, depending upon their specific implementation. Without the promise we're currently making for Persistent storage [this will never be cleared unless you do it or the user

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-12 Thread Charles Pritchard
On Dec 11, 2013, at 10:53 PM, Kinuko Yasuda kin...@chromium.org wrote: persistent storage would behave as in A. I.e. webpages can't use it without there being a prompt involved at some point. Bookmarked webapps can use it without prompt. This sounds kinda reasonable to me, but I'd imagine

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-11 Thread Jonas Sicking
Hi All, Thanks Jan for sending this. Now let me throw a giant wrench into this discussion :-) Unfortunately as we've been discussing webapps, manifests etc at mozilla I've slowly come to the realization that the temporary/persistent categorization isn't really fulfilling all the envisioned use

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-11 Thread Kinuko Yasuda
Hi, I'm glad to hear that IDB's going to support storage types! On Thu, Dec 12, 2013 at 12:39 PM, Jonas Sicking jo...@sicking.cc wrote: Hi All, Thanks Jan for sending this. Now let me throw a giant wrench into this discussion :-) Unfortunately as we've been discussing webapps,

IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-06 Thread Jan Varga
IndexedDB implementation in Firefox 26 (the current beta) supports a new storage type called temporary storage. In short, it's a storage with LRU eviction policy, so the least recently used data is automatically deleted when a limit is reached. Chrome supports something similar [1]. Obviously,

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-06 Thread Joshua Bell
Thanks for sending this! On Fri, Dec 6, 2013 at 7:19 AM, Jan Varga jan.va...@gmail.com wrote: IndexedDB implementation in Firefox 26 (the current beta) supports a new storage type called temporary storage. In short, it's a storage with LRU eviction policy, so the least recently used data

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-06 Thread Boris Zbarsky
On 12/6/13 1:29 PM, Joshua Bell wrote: // Throws TypeError on older implementations since Dictionary won't coerce to Number (?) Sure it will. It'll do ToNumber() and probably end up NaN (which becomes 0 as an unsigned long long) unless your object has a valueOf method that returns

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-06 Thread Joshua Bell
On Fri, Dec 6, 2013 at 11:09 AM, Boris Zbarsky bzbar...@mit.edu wrote: On 12/6/13 1:29 PM, Joshua Bell wrote: // Throws TypeError on older implementations since Dictionary won't coerce to Number (?) Sure it will. It'll do ToNumber() and probably end up NaN (which becomes 0 as an

Re: IndexedDB: Syntax for specifying persistent/temporary storage

2013-12-06 Thread Boris Zbarsky
On 12/6/13 2:37 PM, Joshua Bell wrote: This behavior is specified in prose, not IDL: https://dvcs.w3.org/hg/IndexedDB/raw-file/tip/Overview.html#widl-IDBFactory-open-IDBOpenDBRequest-DOMString-name-unsigned-long-long-version If the value of version is 0 (zero), the implementation must throw a