Closed Bug 425952 Opened 13 years ago Closed 13 years ago

SeaMonkey/MailNews is missing <content/messenger/migration/migration.xul>

Categories

(SeaMonkey :: MailNews: Address Book & Contacts, defect)

defect
Not set
major

Tracking

(Not tracked)

VERIFIED FIXED
seamonkey2.0a1

People

(Reporter: sgautherie, Assigned: iann_bugzilla)

References

Details

(Keywords: regression)

Attachments

(1 file)

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008032900 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

1. Create a new profile.
2. Start Browser.
3. Open Address Book (Ctrl+5).
4. Cancel+Exit "Mailnews" Account Wizard.
5. "Tools > Import...".
6. "Next >".
6r. A "Page Load Error" dialog opens :-(

SeaMonkey/MailNews is missing <content/messenger/migration/migration.xul>,
which is needed since bug 399312 landed:
see
<http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/mailnews/import/resources/content/importDialog.js&rev=1.45&mark=978-979#965>

NB: SeaMonkey/Browser has <content/communicator/migration/migration.xul> (only).

(See +/- related bug 421405 comment 5 and bug 421405 comment 8...)
Flags: blocking-seamonkey2.0a1?
See bug 425956 about viewing the content of the (too small) dialog.
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.2pre) Gecko/2008070302 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

(Bug still there.)
This patch:
* Adds an ifdef to importDialog.js so that it uses the correct file relevant to TB / SM
Assignee: mail → iann_bugzilla
Status: NEW → ASSIGNED
Attachment #334199 - Flags: superreview?(bienvenu)
Attachment #334199 - Flags: review?(bugzilla)
Comment on attachment 334199 [details] [diff] [review]
Add ifdef to importDialog.js so uses correct file for SM/TB (Checkin: Comment 5)

I'd prefer to avoid the ifdefs, but I can't see an easy way around it. so r=me.
Attachment #334199 - Flags: review?(bugzilla) → review+
Attachment #334199 - Flags: superreview?(bienvenu) → superreview+
Comment on attachment 334199 [details] [diff] [review]
Add ifdef to importDialog.js so uses correct file for SM/TB (Checkin: Comment 5)

http://hg.mozilla.org/comm-central/index.cgi/rev/f25b4e8f97a5
Attachment #334199 - Attachment description: Add ifdef to importDialog.js so uses correct file for SM/TB → Add ifdef to importDialog.js so uses correct file for SM/TB (Checkin: Comment 5)
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: blocking-seamonkey2.0a1?
Resolution: --- → FIXED
> I'd prefer to avoid the ifdefs, but I can't see an easy way around it

You could add the following line to /suite/common/jar.mn :
% override chrome://messenger/content/migration/migration.xul chrome://communicator/content/migration/migration.xul
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/20080819081634 SeaMonkey/2.0a1pre] (home, optim default) (W2Ksp4)

V.Fixed
Status: RESOLVED → VERIFIED
Keywords: helpwanted
Flags: in-testsuite-
Clearing in-testsuite-, testing something like this *could* be integrated into Gristmill or some other automated suite.
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.