745508 - Remove unused "From an HTML File" mode from the migration wizard

Status: RESOLVED FIXED

Description

[Mano (::mano, needinfo? for any questions; not reading general bugmail)]

Created attachment 615108 [details] [diff] [review]
patch

I deliberately kept this patch very scoped.

Comment 1

[Mardeg]

Would bug 429583 block this?

Comment 2

[Mano (::mano, needinfo? for any questions; not reading general bugmail)]

I don't think so.  The patch there (from 2009) relies on the old behavior of the places organizer: opening the migration wizard for html-file import.  This has changed in the meanwhile (for quite a long time now, I think), and I'm not sure if the now-dating-again ui-review makes the patch more relevant now, given that.

All in all, I think fixing that bug would involve introducing new UI, independent from the migration wizard.

Comment 3

[Marco Bonardo [::mak]]

Comment on attachment 615108 [details] [diff] [review]
patch

Review of attachment 615108 [details] [diff] [review]:
-----------------------------------------------------------------

Nice cleanup!

This can land with a=desktop-only

Comment 4

[Mano (::mano, needinfo? for any questions; not reading general bugmail)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/e481ab44f21f

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/e481ab44f21f