309093 - Next button state isn't updated after going back

Status: RESOLVED FIXED

(Firefox :: Migration, defect)

Description

[Asaf Romano (gone)]

In the migration wizard, the next button state isn't updated after going back.

STR:
 1. Open the migration wizard, choose one of the options and click Next.
 2. Uncheck everything, at this point, the next button becomes disabled
 3. Go back.

-> The wizard goes back, but the next button isn't re-enabled.

This is a regression from bug 307124, sorry for not catching that on review.

Comment 1

[Mike Connor [:mconnor]]

Attachment: fix

this isn't actually a regression, I can reproduce in 1.0.6

There's still one remaining issue with this sequence, which is that unchecking
all, going back, then going forward rechecks everything.  This is because 0 ==
ALL in nsIBrowserProfileMigrator, but its pretty late for an API change here,
but I'll fix that as a followup patch for trunk only.

http://lxr.mozilla.org/mozilla/source/browser/components/migration/public/nsIBrowserProfileMigrator.idl#48

Comment 2

[Asaf Romano (gone)]

Comment on attachment 196605 [details] [diff] [review]
fix

Please get rid of the whitespcae in the line above onImportItemsPageRewound.

r=mano

Comment 3

[Mike Connor [:mconnor]]

Comment on attachment 196605 [details] [diff] [review]
fix

low-risk polish fix for wizard bustedness

Comment 4

[Mike Connor [:mconnor]]

filed bug Bug 309883 on comment 1

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

*** Bug 303430 has been marked as a duplicate of this bug. ***