rowan/multi-account-containers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #1757 from kafji/kfj/fix-sync-hog

Browse source 
Fix sync resource hog
This commit is contained in:
Andrea Marchesini 2021-10-25 16:59:59 +02:00 • committed by GitHub
commit e830a5bcd8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/js/background/assignManager.js
View file

@ -89,13 +89,13 @@ window.assignManager = {
return;
},
async remove(pageUrlorUrlKey) {
async remove(pageUrlorUrlKey, shouldSync = true) {
const siteStoreKey = this.getSiteStoreKey(pageUrlorUrlKey);
// When we remove an assignment we should clear all the exemptions
this.removeExempted(pageUrlorUrlKey);
await this.area.remove([siteStoreKey]);
const syncEnabled = await this.getSyncEnabled();
if (syncEnabled) await sync.storageArea.backup({siteStoreKey});
if (shouldSync && syncEnabled) await sync.storageArea.backup({siteStoreKey});
return;
},

4
src/js/background/sync.js
View file

@ -493,9 +493,9 @@ async function reconcileSiteAssignments() {
await sync.storageArea.getDeletedSiteList();
for(const siteStoreKey of deletedSiteList) {
if (Object.prototype.hasOwnProperty.call(assignedSitesLocal,siteStoreKey)) {
assignManager
await assignManager
.storageArea
.remove(siteStoreKey);
.remove(siteStoreKey, false);
}
}