1477930 - Remove HasTags

Status: RESOLVED FIXED

(Toolkit :: Places, enhancement, P2)

Description

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

The hastags API is only used by the history result to decide whether to show the expander glyph close to the Tags entry in the Library.
We may not be able to get a synchronous tags status in the future, we could maybe have a delayed cache through a trigger on the new tags table. Anyway, for now we can't do much more than removing this and accepting the glyph may be lying to the user; it will show the Tags section as open-able, but won't show children when opened.
The only alternative would be to make it actually open the container, check for children and close it, but that would execute a synchronous query that'd be pointless work for most of the users.

Comment 1

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

Attachment: Bug 1477930 - Remove taggingService HasTags synchronous API. r=standard8

The tagging API is moving to the bookmarking API, this is part of it.
This will break the tags root open glyph in the Library, for now we don't have
an alternative, we'll re-evaluate in the future in bug 1477934.

Comment 2

[Phabricator Automation]

Comment on attachment 8994501 [details]
Bug 1477930 - Remove taggingService HasTags synchronous API. r=standard8

Mark Banner (:standard8) has approved the revision.

https://phabricator.services.mozilla.com/D2318

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/285d7793a541
Remove taggingService HasTags synchronous API. r=Standard8

Comment 4

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/285d7793a541