Comment 13 for bug 28563

Revision history for this message
In , Gavin Sharp (gavin-sharp) wrote :

Comment on attachment 233006
patch v2.1

>Index: browser/components/migration/content/migration.js

> onImportSourcePageShow: function ()
> {
>+ // Whether or not we offer to import bookmarks from a file
>+ var fromfile = null;

"Reference to the "From File" radio button" would be a better comment, I think.

>+ // The migrator to select. Initialize this to whether or not we can
>+ // import from a file, incase we can't get a migrator source below.
>+ var selectedMigrator = fromfile;

"If the "fromfile" migrator is available, use it as the default in case we have no other migrators"