Remove nsNetscapeProfileMigratorBase.cpp and nsNetscapeProfileMigratorBase.h. Move code needed by Seamonkey migrator into nsSeamonkeyProfileMigrator.cpp.
Created attachment 553208 [details] [diff] [review] Patch
Comment on attachment 553208 [details] [diff] [review] Patch We don't all need to look at this. I'll review it soon.
Comment on attachment 553208 [details] [diff] [review] Patch I don't want to move all this code around. We should get rid of it entirely (by killing the Seamonkey profile migrator). If we're not willing to do that, then we should just leave it as is.
I'd like to remove the Seamonkey profile migrator, if I can. Who should decide about this? Note that removing Seamonkey profile migrator, we can also remove libreg.
Based on the mork removal changes I suspect that migrator is really outdated, btw cc-ing Callek
(In reply to Marco Bonardo [:mak] from comment #5) > Based on the mork removal changes I suspect that migrator is really > outdated, btw cc-ing Callek I bet the migrator is VERY outdated, that said I don't want to completely drop it. The ability to import from similar profiles is a big gain for both Firefox and SeaMonkey. If it is decided (by Module Owner) that the current state of the migrator is bad enough that it must be removed, I'd like an "ok" to re-add it at a future date. And of course, yes, Mork is outdated here, we have Places for Bookmarks and History now, and Sync support.
Does it actually import modern Seamonkey profiles though? I would expect it's importing profiles from the suite days ... which isn't exactly useful.
It imports profiles from a file saved as a Windows registry file. It's completely different from Firefox and, most probably, modern Seamonkey.
(In reply to Marco Castelluccio from comment #8) > It imports profiles from a file saved as a Windows registry file. It's > completely different from Firefox and, most probably, modern Seamonkey. Yes, I just dug through some code to check what it did exactly, and then with my local computer that never had an "old" suite installed. And I can confirm with the supported Suite versions this is dead code. And *very* dead code at that. I'll support simply dropping it at that.
Created attachment 553518 [details] [diff] [review] Remove Seamonkey Profile Migrator
I need someone that could send this to tryserver, before the checkin.
Try run for b291507659ba is complete. Detailed breakdown of the results available here: http://tbpl.allizom.org/?tree=Try&usebuildbot=1&rev=b291507659ba Results (out of 229 total builds): success: 218 warnings: 11 Builds available at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/Ms2ger@gmail.com-b291507659ba