1141547 - [meta] Deprecate synchronous Bookmarks API (Milestone 1)

Status: RESOLVED FIXED

(Toolkit :: Places, defect, P5)

Description

[Tim Taubert [:ttaubert] (inactive)]

Meta bug to track efforts to deprecate the sync Bookmarks API. We can prepare to deprecate but will not be able to actually add deprecation warnings in milestone 1. There are in-tree clients that need to be updated that we can't fix immediately:

1) Add-on SDK (might be the easiest to fix)
2) Sync's bookmarks engine
3) FUEL (that needs to go away)

Comment 1

[Tim Taubert [:ttaubert] (inactive)]

Removing bookmark test conversions from Milestone 1 to narrow the scope of wrapping up the Places work.

Comment 2

[Sylvestre Ledru [:Sylvestre]]

Instead of doing a relnote for bug 1140395, we are going to track this one.

Release Note Request (optional, but appreciated)
[Why is this notable]: Deprecated API, consumers need to update to new API
[Suggested wording]: Places Keywords API deprecated
[Links (documentation, blog post, etc)]:

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Actually, it seems that it is not worth doing a release notes for this.
It is already mentioned here:
https://blog.mozilla.org/addons/2015/06/18/compatibility-for-firefox-40/
which is a more appropriate place for compatibility issues.

Comment 4

[Marco Bonardo [:mak]]

what is mentioned there is just keywords API
the remaining APIs are not yet deprecated for now, hope to get back to these bugs "soon"

Comment 5

[Mark Banner (:standard8)]

With async transactions planned to ship in 58, and bookmarks import now using async bookmark APIs, milestone 1 is now complete.