1319737 - Intermittent errors/crashes when storing wasm modules in indexeddb

Status: RESOLVED DUPLICATE of bug 1319531

(Core :: Storage: IndexedDB, defect, P2)

Description

[Benjamin Bouvier [:bbouvier] (inactive)]

Attachment: test-case.zip

[Tracking Requested - why for this release]: idb issues with wasm caching

STR:
- go to https://benj.me/wasm-idb/

Expected:
- on the first load (creation of IDB and store), sometimes there's an instance of UnknownError during the storeWasm() call, which pretty much creates a new transaction, retrieves the store and calls set(module, 1). It happens in an intermittent fashion and can be reproduced by deleting the store manually in the devtools, then reloading the page.
- reload the page a few times in a row: sometimes the retrieved object from IDB doesn't seem to be a module (it fails in instantiate() by saying that the first argument isn't a wasm module; it is supposed to fail there but only because we don't provide imports during instanciation), sometimes it crashes the entire browser (not only the current tab) and prints this in the (bash) console:

Assertion failure: (mType) == (aType) (unexpected type tag), at /builds/slave/m-cen-l64-ntly-000000000000000/build/src/obj-firefox/ipc/ipdl/_ipdlheaders/mozilla/dom/indexedDB/PBackgroundIDBSharedTypes.h:277

It seems to reproduce more easily when running a local server, so I've attached a zip with the files:
- unzip content
- in the same directory, run: python -m SimpleHTTPServer
- go to http://localhost:8000/test-case.html

Comment 1

[Luke Wagner [:luke]]

The first issue might be bug 1319531.

Comment 2

[Julien Cristau [:jcristau]]

tracking this wasm-related issue for 52

Comment 3

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Hi Janv, any ideas of what's happening here? Does this a blocker of wasm? If so, then I tend to think this is P1.

Comment 4

[Jan Varga [:janv]]

I have no idea why this is happening, but it's definitely a blocker of wasm.

Comment 5

[Andrew Overholt [:overholt]]

Andrew Sutherland said he could take a look here.

Comment 6

[Andrew Sutherland [:asuth] (he/him)]

(Forgot to hit "save changes" when I thought I took this earlier today.  Also looking at bug 1319531 which indeed seems to be the cause of the first load error, and which likely could also explain the second error.  This may end up getting duped to that.)

Comment 7

[Andrew Sutherland [:asuth] (he/him)]

Duping to bug 1319531 with fix for that coming shortly.

Comment 8

[Andrew Sutherland [:asuth] (he/him)]

Clearing tracking status which was propagated to the duped-to in bug 1319531 comment 10.