1502954 - Remove livemarks code from toolkit

Status: RESOLVED FIXED

(Toolkit :: Places, enhancement, P1)

Description

[Marco Bonardo [:mak]]

We removed all the frontend code, now we're left with the toolkit code.
We should understand how to coordinate with suite, we can slightly delay the livemarks code removal, but anyway we want to remove item annotations because they are source of UI jank, so we can't just keep the livemarks code around, it would break regardless when we remove item annotations code.

Jorg, do you have any idea who we should contact, or how we may handle this?

Comment 1

[Marco Bonardo [:mak]]

Note, the change is likely to not break the world, basically livemarks will de-facto become empty folders (though, we can't predict exceptions thrown by the frontend code if it tries to call into no more existing APIs).

Comment 2

[:Gijs (he/him)]

Note that this also blocks removing the SAX XML reader which, as far as we know, has no other in-tree consumers.

TBH, I think it should be fairly straightforward for TB/XM to essentially copy/paste ("move") the toolkit code here and start being responsible for it themselves. Of course, what'll happen when the places support for annotations is removed, I dunno - at least for TB I'd expect it not to care, but suite is probably different.

Comment 3

[Jorg K (CEST = GMT+2)]

(In reply to Marco Bonardo [::mak] from comment #0)
> Jorg, do you have any idea who we should contact, or how we may handle this?
Good question. I work for Thunderbird, and I/we occasionally help out with SeaMonkey stuff. As far as I know, SM trunk isn't a good shape, there have been too many Mozilla platform changes they haven't followed yet, like removal of overlays or bindings.

The guy to talk to is Frank-Rainer.

Comment 4

[Frank-Rainer Grahl (:frg)]

suite builds but is broken. Just remove it. If/when we ever catch up we will fix it.

Comment 5

[Marco Bonardo [:mak]]

I would like to point out this is about Places livemarks code, not the toolkit/components/feeds component, that would require a separate bug, and I guess that's what blocks removing the SAX XML reader?

Comment 6

[:Gijs (he/him)]

(In reply to Marco Bonardo [::mak] from comment #5)
> I would like to point out this is about Places livemarks code, not the
> toolkit/components/feeds component, that would require a separate bug, and I
> guess that's what blocks removing the SAX XML reader?

OK, I've split this off, though I'm not sure if we can proceed with that without first removing the places code that calls into the feed processor...

Comment 7

[Marco Bonardo [:mak]]

that's fine, we'll try to remove the places code in the next few weeks.

Comment 8

[Mark Banner (:standard8)]

Attachment: Bug 1502954 - Remove livemarks code from toolkit. r?mak

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b2654a2729e9
Remove livemarks code from toolkit. r=mak

Comment 10

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/b2654a2729e9

Comment 11

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/b2654a2729e9

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a0668a318ffa
Port bug 1502954 - Remove nsLivemarkService.js from package manifests. rs=bustage-fix