653078 - Init the livemarks service delayed in PlacesCategoriesStarter rather than relying on browser.js

Status: RESOLVED WONTFIX

(Toolkit :: Places, defect)

Description

[Marco Bonardo [:mak]]

While we wait for a overhaul of the service, we should move its initialization in toolkit and use PlacesCategoriesStarter rather then relying on browser.js initing it for us.

Also check if we can make the service init mostly a no-op and delay fetch of all the livemarks to the first update, Sync uses the service for isLivemark and it's possible it's bypassing our delay.

Comment 1

[Marco Bonardo [:mak]]

Attachment: patch v1.0

No perf advantage here, but I think makes sense to move this to Places code, bot from a browser cleanup point of view and to avoid Places users having to figure out how to startup it. In future we may remove start() and handle startup more gracefully.
FWIW, I already fixed Sync code that may init lm service earlier in other patches.

Comment 2

[Dietrich Ayala (:dietrich)]

Comment on attachment 538381 [details] [diff] [review]
patch v1.0

Review of attachment 538381 [details] [diff] [review]:
-----------------------------------------------------------------

profile-after-change occurs how long before delayedStartup() is called?

Comment 3

[Marco Bonardo [:mak]]

(In reply to comment #2)
> profile-after-change occurs how long before delayedStartup() is called?

I don't have timing sadly, but it's a good question. Usually I'd expect it to be ignorable, but clearly it may change if FF starts on a busy system. I guess if it would be safer to bump up delay to 10s.

Comment 4

[Dietrich Ayala (:dietrich)]

I'm not really ok making this change without a more deterministic approach. Especially since there's not a material benefit (ie: perf) making it worth the risk. Yeah, it's nice for other Places consumers, but we know those people, and they know where to find us, so it's not a critical problem.

Comment 5

[Marco Bonardo [:mak]]

Comment on attachment 538381 [details] [diff] [review]
patch v1.0

yeah, considering the no-benefit I'm putting this on sleep for now.

Comment 6

[Marco Bonardo [:mak]]

this is no more needed