Status

()

For bugs in Firefox Desktop, the Mozilla Foundation's web browser. For Firefox user interface issues in menus, bookmarks, location bar, and preferences. Many Firefox bugs will either be filed here or in the Core product. Bugs for developer tools (F12) should be filed in the DevTools product. (more info)

I can get to this sometime in the next 2-3 weeks ... but I'm not sure
that'll be in time for the FF 3.0.1 release. (I'm currently bogged
down in bug 357670 -- not in the patch itself, but contriving tests
for it.)

Feedback from Japanese community tester using following build:
Mozilla/5.0 (Macintosh; U; PPC Mac OS X 10.5; en-US; rv:1.9.1a1pre) Gecko/2008070402 Minefield/3.1a1pre
2843 bookmarks in 64 folders (max depth of the folders is 2) on Safari 3.1.2 was imported correctly including Japanese name folders and their children.
And I also tested with lastest minefield and could be imported some folders from Safari 3.1.1 too.

Note about importing bookmarks from 3.1.2:
I'm still using Safari 3.1.1 and not confirmed yet, but he said that Safari 3.1.2 have new special "Download" folder within the bookmark toolbar and he could not imported from the folder.
Firefox should import form the special "Download" folder too but the special folder was introduced from Safari 3.1.2 and not relating to this bug. It should be handled by different bug.
Thanks.

Sorry, please ignore comment #14. (only #14, not ignore #13)
(In reply to comment #14)
> Note about importing bookmarks from 3.1.2:
>
> I'm still using Safari 3.1.1 and not confirmed yet, but he said that Safari
> 3.1.2 have new special "Download" folder within the bookmark toolbar and he
> could not imported from the folder.
> Firefox should import form the special "Download" folder too but the special
> folder was introduced from Safari 3.1.2 and not relating to this bug.
The special "Download" folder of the tester was hardlink folder made with AppleScript by the tester, himself. So not imported was the correct/natural behavior of Firefox.
All other folders/bookmarks of him are imported correctly and I can confirm the patch works fine.
We hope this will be included in 3.0.2 and later.

fixed on branch:
Checking in browser/components/migration/src/nsSafariProfileMigrator.cpp;
/cvsroot/mozilla/browser/components/migration/src/nsSafariProfileMigrator.cpp,v <-- nsSafariProfileMigrator.cpp
new revision: 1.51; previous revision: 1.50
done
automating the addition of folders/bookmarks into Safari, and then an import from Safari is not trivial. it's very easy to manually test this however, so asking in-litmus.