Closed Bug 605503 Opened 10 years ago Closed 10 years ago

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

Categories

(SeaMonkey :: Build Config, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b2

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(1 file)

No description provided.
Flags: in-testsuite-
Should fix
{
+bin/components/PlacesCategoriesStarter.js
}
Attachment #484353 - Flags: review?(bugspam.Callek)
you should also kill any code that calls PlacesUtils.startPlacesDBUtils() since now the service will start by itself
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.
Attachment #484353 - Flags: review?(bugspam.Callek) → review+
(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.
Component: General → Installer
QA Contact: general → xpi-packages
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 on attachment 484353 [details] [diff] [review]
(Av1) Package PlacesCategoriesStarter.js
[Checked in: Comment 6]

http://hg.mozilla.org/comm-central/rev/4abc23bbf32f
Attachment #484353 - Attachment description: (Av1) Package PlacesCategoriesStarter.js → (Av1) Package PlacesCategoriesStarter.js [Checked in: Comment 6]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Component: Installer → Build Config
Depends on: 431558
QA Contact: xpi-packages → build-config
Resolution: --- → FIXED
I filed bug 605657 for SM 2.0.x.
Depends on: 605657
No longer depends on: 431558
Blocks: 570388
You need to log in before you can comment on or make changes to this bug.