570388 - SeaMonkey doesn't invoke PlacesDBUtils

Status: RESOLVED FIXED

(SeaMonkey :: General, defect)

Description

[:Gavin Sharp [email: [e-mail]]]

I don't actually know whether this is a problem or not, but I noticed that SeaMonkey does not invoke PlacesDBUtils, while Firefox does. Should it?

(see also bug 570387, which would make this moot)

Comment 1

[Robert Kaiser]

I'm also not sure if it is a problem, need to investigate what it even does, but the relevant line is in delayedStartup(), see http://mxr.mozilla.org/comm-central/source/mozilla/browser/base/content/browser.js#1383

How did you find out about that, just out of interest?

Comment 2

[:Gavin Sharp [email: [e-mail]]]

I was looking into it while filing bug 570387. Firefox was the only toolkit user invoking this code, which seemed odd to me. Potentially a problem for Fennec too, I guess.

Comment 3

[Marco Bonardo [:mak]]

it was added originally to address corruption issues in alpha versions of Firefox, and that explains why projects integrating Places later do not have it.
Bug 570387 would solve the FX-only issue, but i'm not sure all implementers will want it, it runs on idle and can execute some heavy task, not something we can do on mobile.
Sure, right now it could also help finding some corruption caused by weave or other extensions, so it is sensible.

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #0)
> (see also bug 570387, which would make this moot)

(FF) Bug 570387 was ported in (SM) bug 605503 :-)