605503 - Port |Bug 570387 - PlacesDBUtils should register itself in the idle-daily category| to SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect)

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Package PlacesCategoriesStarter.js [Checked in: Comment 6]

Should fix
{
+bin/components/PlacesCategoriesStarter.js
}

Comment 2

[Marco Bonardo [:mak]]

you should also kill any code that calls PlacesUtils.startPlacesDBUtils() since now the service will start by itself

Comment 3

[Justin Wood (:Callek)]

Comment on attachment 484353 [details] [diff] [review]
(Av1) Package PlacesCategoriesStarter.js
[Checked in: Comment 6]

Without checking package-compare or stuff, I'll treat it as a rs+. but I'm comfortable here.

Comment 4

[Justin Wood (:Callek)]

(In reply to comment #2)
> you should also kill any code that calls PlacesUtils.startPlacesDBUtils() since
> now the service will start by itself

FYI: http://mxr.mozilla.org/comm-central/search?string=startPlacesDBUtils returns no comm-central users.

Comment 5

[Marco Bonardo [:mak]]

most likely it was never activated on SM. This change makes that code more toolkit-ish (Apart the packaging), so it should be better.

Comment 6

[Serge Gautherie (:sgautherie)]

Comment on attachment 484353 [details] [diff] [review]
(Av1) Package PlacesCategoriesStarter.js
[Checked in: Comment 6]

http://hg.mozilla.org/comm-central/rev/4abc23bbf32f

Comment 7

[Serge Gautherie (:sgautherie)]

I filed bug 605657 for SM 2.0.x.