let's do this.
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.
Created attachment 711118 [details] [diff] [review] patch v1.1 Forgot to update PlacesUtils.livemarks
Comment on attachment 711118 [details] [diff] [review] patch v1.1 not a lot to say, just nsILivemarkService interface going away
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).