Skip to content
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

Implement local storage infrastructure #244

Closed
brson opened this issue Jan 5, 2013 · 5 comments
Closed

Implement local storage infrastructure #244

brson opened this issue Jan 5, 2013 · 5 comments

Comments

@brson
Copy link
Contributor

brson commented Jan 5, 2013

Servo doesn't have any local storage. This will be needed for history, bookmarks, caching and a million other things. At this time we are inclined to use LevelDB for everything, so a good place to start would be LevelDB bindings. Then move on to some of the common browser features and figure out how to integrate them into the engine.

@amuxtux
Copy link

amuxtux commented Jan 8, 2013

Should we consider security features at the moment (e.g. secure storage)

@brson
Copy link
Contributor Author

brson commented Jan 8, 2013

In my opinion we should design with security and privacy in mind, but we don't necessarily need to implement security features yet.

@brson
Copy link
Contributor Author

brson commented Jan 8, 2013

I just noticed some LevelDB bindings here. Might be a good starting point.

ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
@Manishearth
Copy link
Member

triage: We have session storage, and a session storage object that pretends to be local storage

@jdm
Copy link
Member

jdm commented Dec 8, 2015

The features described here aren't related to sessionStorage. I don't see a reason to keep this open right now. We should discuss persistent storage requirements as needed.

@jdm jdm closed this as completed Dec 8, 2015
jdm pushed a commit to jdm/servo that referenced this issue Mar 6, 2021
Make the build warning-free.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/244)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants