+++ This bug was initially created as a clone of Bug #753205 +++ (In reply to Marco Bonardo [:mak] from bug 739041 comment #35) > (In reply to Serge Gautherie (:sgautherie) from bug 739041 comment #34) > > Marco, can you confirm that it is expected that the separator is not > > imported (anymore?)? > > The separators should be imported afaik, it's their names we don't import > anymore, but this one doesn't even have a name... > The firefox bookmarks preplaces doesn't have a separator, so there may even > be an actual regression here, indeed the old code had an handleSeparator, > the new code is missing it :( Bug 753205 broke a bunch of our places tests (Such as test_384370.js), due in part to http://hg.mozilla.org/mozilla-central/diff/904364bcee0e/toolkit/components/places/tests/head_common.js e.g. e:/builds/slave/test/build/xpcshell/tests/suite/common/places/tests/unit/head_bookmarks.js -> file:///e:/builds/slave/test/build/xpcshell/tests/toolkit/components/places/tests/head_common.js:408: TypeError: redeclaration of const FILENAME_BOOKMARKS_HTML

Comment on attachment 639123 [details] [diff] [review] (Av1) Port |Bug 753205 - bookmarks import ignores separators| to SeaMonkey [Checked in: Comment 3] http://hg.mozilla.org/comm-central/rev/881420f9c131

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1341688685.1341690194.4905.gz WINNT 5.2 comm-central-trunk debug test xpcshell on 2012/07/07 12:18:05 V.Fixed. NB: There is a "test_browserGlue_smartBookmarks.js | 6 == 5" error, but ftb I assume it is not this bug.

Comment on attachment 639123 [details] [diff] [review] (Av1) Port |Bug 753205 - bookmarks import ignores separators| to SeaMonkey [Checked in: Comment 3] Test only, not approving