User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9) Gecko/2008061004 Firefox/3.0 Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9) Gecko/2008061004 Firefox/3.0 If a folder in Safari bookmarks has a name with non uniform characters (such as a non-English letters language), Firefox will not import it over. This appears to only be in OS-X from Safari. Created as spin off of bug 417264. Reproducible: Always Steps to Reproduce: 1.Create a folder in Safari with accents or Chinese/Japanese characters. 2.Open up Firefox to import bookmarks. 3.Import bookmarks from Safari. Actual Results: Bookmarked folder will not import. Expected Results: Bookmarked folder will import.
In Japan, 40% of Mac users use Firefox. We'd like to see the fix this ASAP.
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.)
Steven, thanks for your feedback. Could we have another Mac-driver look at this in hopes that we could get this in to 3.0.1?
Likely, this needs the same treatment as bug 417264. I can fix this tomorrow... unless someone else gets to it first :)
Created attachment 326348 [details] [diff] [review] fix
safari folders w/ intl chars in the title import successfully w/ the attached fix.
Dietrich: got any tests? Mano: please nominate for approval184.108.40.206 after reviewing.
Comment on attachment 326348 [details] [diff] [review] fix r=mano
Do we have any automated tests for this? If not, I can gather some folks to help test. Please let me know. I think we're too late for 3.0.1?
If we can have a test build posted here, we can help test.
(In reply to comment #10) > If we can have a test build posted here, we can help test. Here is a test build including the patch built with tryserver: https://build.mozilla.org/tryserver-builds/2008-07-01_04:email@example.comfirstname.lastname@example.org And I can import bookmarks within Japanese name folders with this test build. (In reply to comment #9) > I think we're too late for 3.0.1? Beltzner have already changed the flag to the patch before your comment, which means we're too late for 3.0.1
fixed in mozilla-central (46c6d7797006)
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.
Comment on attachment 326348 [details] [diff] [review] fix Approved for 220.127.116.11. Please land in CVS. a=ss It really would be good to get some automated tests for this though...
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.
verified with: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:18.104.22.168) Gecko/2008092414 Firefox/3.0.3 and Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9.1b1pre) Gecko/20080930022236 Minefield/3.1b1pre