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

Status: VERIFIED FIXED

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

Description

[Serge Gautherie (:sgautherie)]

[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...)

Comment 1

[Serge Gautherie (:sgautherie)]

See bug 425956 about viewing the content of the (too small) dialog.

Comment 2

[Serge Gautherie (:sgautherie)]

[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.)

Comment 3

[Ian Neal]

Attachment: Add ifdef to importDialog.js so uses correct file for SM/TB (Checkin: Comment 5)

This patch:
* Adds an ifdef to importDialog.js so that it uses the correct file relevant to TB / SM

Comment 4

[Mark Banner (:standard8)]

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.

Comment 5

[Ian Neal]

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

Comment 6

[Philip Chee]

> 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

Comment 7

[Serge Gautherie (:sgautherie)]

[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

Comment 8

[Mark Banner (:standard8)]

Clearing in-testsuite-, testing something like this *could* be integrated into Gristmill or some other automated suite.