diff options
author | toasted-nutbread <toasted-nutbread@users.noreply.github.com> | 2021-08-07 15:25:00 -0400 |
---|---|---|
committer | toasted-nutbread <toasted-nutbread@users.noreply.github.com> | 2021-08-08 16:04:57 -0400 |
commit | 4341b5078343dccc3b082a496fd6c158b872fc99 (patch) | |
tree | 2e175f3f25c4ef14438fe9a28f7c8fe9a1b6d549 | |
parent | b590a147015df21eec02054eb542ff23f2a4c0ef (diff) |
Fix data not being persisted after terminating Worker thread (#1877)
-rw-r--r-- | ext/js/data/database.js | 10 | ||||
-rw-r--r-- | ext/js/language/dictionary-database.js | 4 | ||||
-rw-r--r-- | ext/js/language/dictionary-importer.js | 4 |
3 files changed, 18 insertions, 0 deletions
diff --git a/ext/js/data/database.js b/ext/js/data/database.js index a0a1804a..222e376c 100644 --- a/ext/js/data/database.js +++ b/ext/js/data/database.js @@ -198,6 +198,16 @@ class Database { }); } + persistData(objectStoreName) { + return new Promise((resolve, reject) => { + const transaction = this.transaction([objectStoreName], 'readonly'); + const objectStore = transaction.objectStore(objectStoreName); + const cursor = objectStore.openCursor(); + cursor.onerror = (e) => reject(e.target.error); + cursor.onsuccess = () => resolve(); + }); + } + static deleteDatabase(databaseName) { return new Promise((resolve, reject) => { const request = indexedDB.deleteDatabase(databaseName); diff --git a/ext/js/language/dictionary-database.js b/ext/js/language/dictionary-database.js index 6b235fb6..62300676 100644 --- a/ext/js/language/dictionary-database.js +++ b/ext/js/language/dictionary-database.js @@ -317,6 +317,10 @@ class DictionaryDatabase { return this._db.bulkAdd(objectStoreName, items, start, count); } + persistData(objectStoreName) { + return this._db.persistData(objectStoreName); + } + // Private _findMultiBulk(objectStoreName, indexNames, items, createQuery, predicate, createResult) { diff --git a/ext/js/language/dictionary-importer.js b/ext/js/language/dictionary-importer.js index 89417ca6..8d6dcb33 100644 --- a/ext/js/language/dictionary-importer.js +++ b/ext/js/language/dictionary-importer.js @@ -149,6 +149,10 @@ class DictionaryImporter { this._progressData.index += count; this._progress(); } + + // This function is required in order to make the added data persist after the worker is terminated. + // https://bugs.chromium.org/p/chromium/issues/detail?id=1237686 + await dictionaryDatabase.persistData(objectStoreName); }; await bulkAdd('terms', termList); |