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

Content bug: Remove references to caching WebAssembly Modules in indexedDB #14520

Closed
evilpie opened this issue Mar 31, 2022 · 3 comments · Fixed by #22141
Closed

Content bug: Remove references to caching WebAssembly Modules in indexedDB #14520

evilpie opened this issue Mar 31, 2022 · 3 comments · Fixed by #22141
Labels
Content:wasm WebAssembly docs effort: small This task is a small effort. help wanted If you know something about this topic, we would love your help!

Comments

@evilpie
Copy link
Contributor

evilpie commented Mar 31, 2022

What page(s) did you find the problem on?

https://developer.mozilla.org/en-US/docs/WebAssembly/Caching_modules

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/WebAssembly/instantiate

shared via postMessage() or cached in IndexedDB.

There are probably more references, probably best to search for indexedDB under WebAssembly/

Specific page section or heading?

What is the problem?

IndexedDB caching was removed from Firefox 4 years ago, I don't think we need to keep the documentation for it. https://bugzilla.mozilla.org/show_bug.cgi?id=1469395

What did you expect to see?

Did you test this? If so, how?

@evilpie evilpie added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Mar 31, 2022
@sideshowbarker sideshowbarker added Content:JS JavaScript docs Content:wasm WebAssembly docs help wanted If you know something about this topic, we would love your help! effort: small This task is a small effort. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Mar 31, 2022
@wbamberg
Copy link
Collaborator

So this whole page should be removed:

https://developer.mozilla.org/en-US/docs/WebAssembly/Caching_modules

...is that right?

(as well as some other bits of cleanup)

@evilpie
Copy link
Contributor Author

evilpie commented Apr 1, 2022

I think we can just delete whole page. I am not sure who we could ask for details.

@hamishwillee
Copy link
Collaborator

I'm late to the party, but IMO the page is out of date but the use case is not. It would be better to replace this with a more recent caching mechanism.

So would suggest finding someone who could help do that. If not, maybe deleting the page and leaving it as a redlink?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:wasm WebAssembly docs effort: small This task is a small effort. help wanted If you know something about this topic, we would love your help!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants