1833427 - Add a Bookmarks file import option to the new migration wizard

Status: RESOLVED FIXED

(Firefox :: Migration, task, P2)

Description

[Mike Conley (:mconley) (:⚙️)]

This bug is for adding a new FileMigrator to https://searchfox.org/mozilla-central/rev/f60cf6bfa8bd096efd9bb3a445364f5a0f32897a/browser/components/migration/FileMigrators.sys.mjs for importing bookmarks from an HTML file (and perhaps also a JSON file, depending on the file type being imported)...

This will mostly just be wrapping the importing capability from https://searchfox.org/mozilla-central/rev/f60cf6bfa8bd096efd9bb3a445364f5a0f32897a/toolkit/components/places/BookmarkHTMLUtils.sys.mjs and https://searchfox.org/mozilla-central/rev/f60cf6bfa8bd096efd9bb3a445364f5a0f32897a/toolkit/components/places/BookmarkJSONUtils.sys.mjs.

See the Password CSV file importer for an example of how to implement this: https://searchfox.org/mozilla-central/rev/f60cf6bfa8bd096efd9bb3a445364f5a0f32897a/browser/components/migration/FileMigrators.sys.mjs#157-247

The new migrator will need to be added here too: https://searchfox.org/mozilla-central/rev/f60cf6bfa8bd096efd9bb3a445364f5a0f32897a/browser/components/migration/MigrationUtils.sys.mjs#111-115

Comment 1

[Mark Hammond [:markh] [:mhammond]]

FYI, bug 1419545 reports that mobile bookmarks aren't exported when exporting as HTML (but are as JSON), which might be relevant here.

Comment 2

[Tim Giles [:tgiles] | OOO Nov 25-29]

Attachment: Bug 1833427 - Add bookmark import from HTML file in new migration wizard. r?kpatenio!

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Commandeering per discussion with tgiles. Thanks for getting this so far, Tim!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8dee8a344f4f
Add bookmark import from HTML file in new migration wizard. r=fluent-reviewers,kpatenio,flod

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/8dee8a344f4f