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.

Sign up for GitHub

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

Jump to bottom

Remove navigator.yieldForStorageUpdates() #334

Closed
foolip opened this issue Nov 13, 2015 · 4 comments
Closed

Remove navigator.yieldForStorageUpdates() #334

foolip opened this issue Nov 13, 2015 · 4 comments
Labels
removal/deprecation Removing or deprecating a feature

Comments

@foolip
Copy link
Member

foolip commented Nov 13, 2015

https://html.spec.whatwg.org/#dom-navigator-yieldforstorageupdates

Nobody has implemented it, and it's likely anybody ever will, because the storage mutex concept itself hasn't been implemented.
@foolip foolip added the removal/deprecation Removing or deprecating a feature label Nov 13, 2015
@domenic
Copy link
Member

domenic commented Nov 13, 2015

As sad as it is, we should probably also remove the storage mutex, replacing it with warnings and examples of how cookies and local/session storage are all highly unsafe due to its lack. And maybe suggesting alternatives, if there are any...

@foolip
Copy link
Member Author

foolip commented Nov 13, 2015

Yes, we should probably do that as well. Want to lump that into this issue, or a separate one?

@domenic
Copy link
Member

domenic commented Nov 13, 2015

I'll open a new one to contain any controversy there. Removing an unimplemented public API is probably less controversial...

@domenic domenic mentioned this issue Nov 13, 2015
Closed
foolip added a commit that referenced this issue Nov 14, 2015
@foolip
Remove navigator.yieldForStorageUpdates() due to lack of implementation
dbe6cc8 
Also rename NavigatorStorageUtils to NavigatorCookies.

Fixes #334
@foolip foolip mentioned this issue Nov 14, 2015
Closed
foolip added a commit that referenced this issue Nov 16, 2015
@foolip
Remove navigator.yieldForStorageUpdates() due to lack of implementation
894d2e5 
Also rename NavigatorStorageUtils to NavigatorCookies.

Fixes #334
@foolip foolip changed the title Remove navigator. yieldForStorageUpdates() Remove navigator.yieldForStorageUpdates() Nov 16, 2015
@foolip
Copy link
Member Author

foolip commented Nov 16, 2015

Closing this in favor of #335 to keep discussion in a single place.

@foolip foolip closed this as completed Nov 16, 2015
foolip added a commit that referenced this issue Nov 24, 2015
@foolip
Remove navigator.yieldForStorageUpdates() due to lack of implementation
3f7874a 
Also rename NavigatorStorageUtils to NavigatorCookies.

Fixes #334
foolip added a commit that referenced this issue Dec 15, 2015
@foolip
Remove navigator.yieldForStorageUpdates() due to lack of implementation
eff2e1b 
Also rename NavigatorStorageUtils to NavigatorCookies.

Fixes #334
foolip added a commit that referenced this issue Dec 15, 2015
@foolip
Remove navigator.yieldForStorageUpdates() due to lack of implementation
c6fb03d 
Also rename NavigatorStorageUtils to NavigatorCookies.

Fixes #334
@ayg ayg mentioned this issue Aug 6, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
removal/deprecation Removing or deprecating a feature
Milestone
No milestone
Development

No branches or pull requests
2 participants
@foolip @domenic