Bug 1890427 - Part 3: Use a PlacesObserver to regenerate backups when a bookmark is deleted. r=backup-reviewers,kpatenio

Differential Revision: https://phabricator.services.mozilla.com/D218886
This commit is contained in:
Mike Conley
2024-09-18 13:32:08 +00:00
parent 67293d208f
commit 5e0572f68f
2 changed files with 39 additions and 2 deletions

View File

@@ -3155,7 +3155,7 @@ export class BackupService extends EventTarget {
this.onPlacesEvents.bind(this)
);
PlacesObservers.addListener(
["history-cleared", "page-removed"],
["history-cleared", "page-removed", "bookmark-removed"],
this.#placesObserver
);
@@ -3182,7 +3182,7 @@ export class BackupService extends EventTarget {
);
PlacesObservers.removeListener(
["history-cleared", "page-removed"],
["history-cleared", "page-removed", "bookmark-removed"],
this.#placesObserver
);
@@ -3323,6 +3323,8 @@ export class BackupService extends EventTarget {
}
break;
}
case "bookmark-removed":
// Intentional fall-through
case "history-cleared": {
this.#debounceRegeneration();
return;