Dictionary delete improvements (#844)

* Change where clearDatabaseCaches occurs

* Move dictionary deletion into the settings page

* Remove api.deleteDictionary

1 files changed, 1 insertions, 8 deletions

diff --git a/ext/bg/js/backend.js b/ext/bg/js/backend.js
index 54982098..f656adbd 100644
--- a/ext/bg/js/backend.js
+++ b/ext/bg/js/backend.js
@@ -118,7 +118,6 @@ class Backend {
             ['triggerDatabaseUpdated',       {async: false, contentScript: true,  handler: this._onApiTriggerDatabaseUpdated.bind(this)}]
         ]);
         this._messageHandlersWithProgress = new Map([
-            ['deleteDictionary',        {async: true,  contentScript: false, handler: this._onApiDeleteDictionary.bind(this)}]
         ]);
 
         this._commandHandlers = new Map([
@@ -700,7 +699,6 @@ class Backend {
     }
 
     async _onApiPurgeDatabase() {
-        this._translator.clearDatabaseCaches();
         await this._dictionaryDatabase.purge();
         this._triggerDatabaseUpdated('dictionary', 'purge');
     }
@@ -742,12 +740,6 @@ class Backend {
         return details;
     }
 
-    async _onApiDeleteDictionary({dictionaryName}, sender, onProgress) {
-        this._translator.clearDatabaseCaches();
-        await this._dictionaryDatabase.deleteDictionary(dictionaryName, {rate: 1000}, onProgress);
-        this._triggerDatabaseUpdated('dictionary', 'delete');
-    }
-
     async _onApiModifySettings({targets, source}) {
         const results = [];
         for (const target of targets) {
@@ -1721,6 +1713,7 @@ class Backend {
     }
 
     _triggerDatabaseUpdated(type, cause) {
+        this._translator.clearDatabaseCaches();
         this._sendMessageAllTabs('databaseUpdated', {type, cause});
     }