Closed Bug 838798 Opened 7 years ago Closed 7 years ago
Remove deprecated ns
ILivemark Service interface
let's do this.
Summary: Remove deprecated Livemarks interface → Remove deprecated nsILivemarkService interface
and also PlacesUtils.itemIsLivemark PlacesUtils.nodeIsLivemarkContainer PlacesUtils.nodeIsLivemarkItem
Comment on attachment 710956 [details] [diff] [review] patch v1.0 Ok, I think I got all of the pieces, will send to Try before landing, just in case.
Forgot to update PlacesUtils.livemarks
Attachment #711118 - Flags: review?(mano) → review+
Comment on attachment 711118 [details] [diff] [review] patch v1.1 not a lot to say, just nsILivemarkService interface going away
Attachment #711118 - Flags: superreview?(gavin.sharp)
Have we shipped deprecation warnings for this interface? I see a lot of users on AMO (both directly and via PlacesUtils).
yes, all methods of the interface were sending messages to the console about deprecation, plus the interface itself said it was deprecated and to use the async version. I honestly don't think we are really lucky with add-ons conversion to new APIs until they actually hit the API removal (for various reasons, even valid, like the fact most are volunteer with few time).
Attachment #711118 - Flags: superreview?(gavin.sharp) → superreview+
Target Milestone: --- → mozilla22
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.