IndexedDB: support async wasm compilation in chrome upgrade cases |
||||||||
Issue descriptionThis supports carrying out compiling wasm bytes in cases when deserialization wouldn't work (because Chrome upgrade) asynchronously, before onsuccess is called.
,
Apr 12 2017
,
Apr 17 2017
,
Apr 17 2017
(forgot to add) This also covers round-tripping the bytes back to storage - not sure if that constitutes a separate workitem from IDB's perspective.
,
Apr 18 2017
,
Apr 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ca74aee3773bb77867cf02d5f81725d10b33e4ab commit ca74aee3773bb77867cf02d5f81725d10b33e4ab Author: dmurph <dmurph@chromium.org> Date: Fri Apr 28 02:09:39 2017 [IndexedDB] Hold referenced to IDB::put blobs in the IDBRequest Since blobs can die during transit (pending mojo change), we can prevent this by holding onto the blob handles in the IDBRequest until we get the response back from the browser. R=pwnall,jsbell BUG= 351753 , 710736 , 713409 Review-Url: https://codereview.chromium.org/2828953002 Cr-Commit-Position: refs/heads/master@{#467836} [modify] https://crrev.com/ca74aee3773bb77867cf02d5f81725d10b33e4ab/third_party/WebKit/Source/modules/indexeddb/IDBObjectStore.cpp [modify] https://crrev.com/ca74aee3773bb77867cf02d5f81725d10b33e4ab/third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp [modify] https://crrev.com/ca74aee3773bb77867cf02d5f81725d10b33e4ab/third_party/WebKit/Source/modules/indexeddb/IDBRequest.h
,
Apr 30 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 1 2018
Assuming we still want this some day. Perhaps we should mark this 'ExternalDependency' ?
,
Dec 12
Archiving (maybe should close?)
,
Dec 13
Agree, closing. We decided not to do IndexedDB support for caching WASM modules. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by jsb...@chromium.org
, Apr 12 2017