for each examinated id it does: // Make sure to remove items that have the exclude annotation. if (Svc.Annos.itemHasAnnotation(itemId, EXCLUDEBACKUP_ANNO)) { this.removeChangedID(this._GUIDForId(itemId)); return true; } itemHasAnnotation is a query to the database, since excluded items are a really small number, we should rather add a cached getter in PlacesUtils, like we did for readonly annotation (see PlacesUtils._readOnly), and then query through a PlacesUtils.itemIsReadOnlyFolder(itemId);

ehr, I meant PlacesUtils.itemIsExcludedFromBackup(itemId);

mak: I'm putting a dep on anno lookup improvements, but do you have a bug for the work you suggest in Comment 0?

nope, though it's easy to do, just copying http://mxr.mozilla.org/mozilla-central/source/toolkit/components/places/PlacesUtils.jsm#239 to an _excludeFromBackup and exposing a PlacesUtils.shouldExcludeFromBackup(aItemId) that uses _excludeFromBackup internally.

`excludeFromBackup` is no longer a thing.