fix(storage): Add checkStorage method to freeStorage regularly

Signed-off-by: Marcel Klehr <[email protected]>

Author: marcelklehr

src/lib/browser/BrowserController.js

@@ -22,6 +22,10 @@ class AlarmManager {
     this.ctl = ctl
   }
 
+  async checkStorage() {
+    await freeStorageIfNecessary()
+  }
+
   async checkSync() {
     const accounts = await BrowserAccountStorage.getAllAccounts()
     for (let accountId of accounts) {
@@ -105,6 +109,7 @@ export default class BrowserController {
 
     // Set up the alarms
 
+    browser.alarms.create('checkStorage', { periodInMinutes: 15 })
     browser.alarms.create('checkSync', { periodInMinutes: 1 })
     browser.alarms.onAlarm.addListener(async alarm => {
       await this.alarms[alarm.name]()
@@ -129,7 +134,6 @@ export default class BrowserController {
       },
       []
     )
-    freeStorageIfNecessary()
 
     // do some cleaning if this is a new version
 

src/lib/native/NativeController.js

@@ -15,6 +15,7 @@ class AlarmManager {
     this.ctl = ctl
     this.backgroundSyncEnabled = true
     setInterval(() => this.checkSync(), 25 * 1000)
+    setInterval(() => this.checkStorage(), 30 * 1000)
 
     Network.addListener('networkStatusChange', status => {
       if (status.connected) {
@@ -25,6 +26,10 @@ class AlarmManager {
     })
   }
 
+  async checkStorage() {
+    await freeStorageIfNecessary()
+  }
+
   async checkSync() {
     if (!this.backgroundSyncEnabled) {
       return
@@ -93,7 +98,6 @@ export default class NativeController {
       },
       []
     )
-    freeStorageIfNecessary()
 
     // lock accounts when locking is enabled