Created attachment 344474 [details] [diff] [review]
I wanted to drop the current (unnecessary) nsIPasswordManagerInternal.h includes in mail/components/migration, but then I thought there may be some others, so I did a bit of digging, and removed a few extra.
This is just a very small part of the password manager migration/cleanup.
I've tested it on my mac, and on linux (Linux builds with frozen API in that dir) and it seems to work fine.
Comment on attachment 344474 [details] [diff] [review]
I'm curious why we're still building the dogbert stuff - I believe that's migration from 4.x, and I thought we'd dropped that.
I remember that being discussed, but the import wizard still offers it.
(ran into it when running litmus on a3 where it promptly crashed)
(In reply to comment #1)
> (From update of attachment 344474 [details] [diff] [review])
> I'm curious why we're still building the dogbert stuff - I believe that's
> migration from 4.x, and I thought we'd dropped that.
kill 4.x import did not git finished in bug 379068, "Remove support for Netscape 4.x migration from the trunk". Discussed with bienvenu on IRC that 4.x *import* should indeed be killed. Do we reopen 379068, or do we handle it in this or a new bug? Just need to ensure it doesn't impact imports from newer releases.
(And remove 4.x import from litmus tests. Check if 6.x and 7.x import is in litmus tests)
I checked this in a few days ago. Various discussions on import capabilities have been started elsewhere.