Closed Bug 699844 (removeAnnotations) Opened 13 years ago Closed 4 years ago

Remove the Annotations Service API

Categories

(Toolkit :: Places, task, P3)

Product:
Toolkit
Component:
Places
Type:
task
Points:
3

Tracking

()

Status:
RESOLVED FIXED
Milestone:
84 Branch
Iteration:
84.1 - Oct 19 - Nov 01
Tracking Flags:
Tracking Status
firefox84 --- fixed

People

(Reporter: mak, Assigned: standard8)

References

(Depends on 1 open bug, Blocks 3 open bugs)

Details

Attachments

(3 files, 1 obsolete file)

Bug 699844 - Remove remaining uses of bookmark annotations from Sync. r?markh!
(deleted), text/x-phabricator-request
Details
Bug 699844 - Remove the live bookmark migrator as it is not needed now. r?mak!,markh!
(deleted), text/x-phabricator-request
Details
Bug 699844 - Remove the Bookmarks annotations service API. r?mak!
(deleted), text/x-phabricator-request
Details
To avoid concurrency issues this requires its users to be async, at least those history and bookmarks parts depending on it.
Alias: asyncAnnotations
Depends on: placesAsyncBookmarks
Depends on: 822996
No longer depends on: placesAsyncBookmarks
Blocks: OMTPlaces
No longer blocks: StorageJank
Blocks: 1071511
Priority: -- → P3
Depends on: 1387446
Depends on: 1072833
We are re-evaluating this as a whole. We'll likely first completely remove item annotations, and then re-evaluate an approach for page annotations in the few points where we really need them. Bugs incoming.
Depends on: 1460577
Depends on: 1462135
Attachment #767924 - Attachment is obsolete: true
Depends on: 1468972
Summary: Make Annotations Service APIs asynchronous → Remove the Annotations Service API
Alias: asyncAnnotations → removeAnnotations
Depends on: 1467131
No longer depends on: 1072833, 1462135
Depends on: 1475245
Depends on: 1475558
Blocks: 1475582
Depends on: 1479053
Depends on: 1479764
No longer depends on: 1387446
Assignee: nobody → standard8

Depends on D93314

Severity: normal → N/A
Type: defect → task
Iteration: --- → 84.1 - Oct 19 - Nov 01
Points: --- → 3
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2c2fd8e25f29 Remove remaining uses of bookmark annotations from Sync. r=markh https://hg.mozilla.org/integration/autoland/rev/413a70305cc6 Remove the live bookmark migrator as it is not needed now. r=mak,markh https://hg.mozilla.org/integration/autoland/rev/61283442e9fe Remove the Bookmarks annotations service API. r=mak

https://hg.mozilla.org/mozilla-central/rev/2c2fd8e25f29
https://hg.mozilla.org/mozilla-central/rev/413a70305cc6
https://hg.mozilla.org/mozilla-central/rev/61283442e9fe

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox84: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: