Look at the code that imports from other browsers and update it to use the new Bookmarks/History APIs if necessary using #ifdef MOZ_PLACES for now. Test import from various browsers in the following cases: "warm import" into an existing Firefox profile "cold import" (move aside all your firefox profiles, then run firefox invoking auto-import) There is quite a test matrix here, just look at all the migrators!
13 years ago
Severity: normal → blocker
Priority: -- → P1
Created attachment 209500 [details] [diff] [review] Uses new bookmarks service for Migration I've tested this on Windows with Opera 8.5, IE 6.0, and Netscape 4.8. Will test on Mac w/Safari before checking in.
Created attachment 209649 [details] [diff] [review] Fixes typo in safari migrator and includes missing idl file
Comment on attachment 209649 [details] [diff] [review] Fixes typo in safari migrator and includes missing idl file email@example.com
Attachment #209649 - Flags: review?(bugs) → review+
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
For 1.8branch, NS_ConvertUTF8toUTF16 shouldn't have been used. keyNameStr comes from a Windows 'A' API so that NS_CopyNativeToUnicode should be used instead. (Perhaps, we have to add this point to our string document) #ifdef MOZ_PLACES bms->InsertItem(keywordsFolder, uri, -1); bms->SetItemTitle(uri, NS_ConvertUTF8toUTF16(keyNameStr)); #else This point got moot by 'migrating' the migrator code to use nsWindowsRegkey in bug 250255, but I thought it might be a good idea to note that here to minimize the number of similar mistakes in the future.
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.