-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add KV Storage Built In Module #1923
Comments
Actually reading the spec further, it is actually a standard modern API layer on top of IndexDB and uses IndexDB to persist the storage. So #1699 is a requirement before we could implement this. |
Doesn't mean that we have to implement it like that. |
Well, if you look at the spec, there is a expected back off to IndexedDB where you can gain access to the db instance for advanced usage. |
They way you put it the first time it sounded like it was just a implementation choice, but no it actually exists in the spec https://wicg.github.io/kv-storage/#storagearea-backingstore. |
Well that is part of the IndexDB spec. |
I meant the KV Storage implementations not the browser itself. |
That is also in the draft spec... Like the ordering of keys, etc. There are a couple concerns regarding this layering though which are also addressed in the draft spec: https://wicg.github.io/kv-storage/#perfect-polyfill |
No browsers now intend to implement this, so work is suspended: https://github.com/WICG/kv-storage#status We shouldn't consider implementing until there is further movement. |
In Chromium 74, KV Storage has been implemented as a built-in module. The module is a far more rational API than IndexDB (and local storage).
The WICG draft is located here: https://wicg.github.io/kv-storage/
This would require #1922
This could negate the need for #1699
The text was updated successfully, but these errors were encountered: