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

VERIFIED FIXED in seamonkey2.0a1

Status

--
major
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: sgautherie, Assigned: iann_bugzilla)

Tracking

({regression})

Trunk
seamonkey2.0a1
regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
[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?
(Reporter)

Comment 1

11 years ago
See bug 425956 about viewing the content of the (too small) dialog.
(Reporter)

Comment 2

11 years ago
[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.)
(Assignee)

Comment 3

11 years ago
Created attachment 334199 [details] [diff] [review]
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
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+

Updated

11 years ago
Attachment #334199 - Flags: superreview?(bienvenu) → superreview+
(Assignee)

Comment 5

11 years ago
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)
(Assignee)

Updated

11 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Flags: blocking-seamonkey2.0a1?
Resolution: --- → FIXED

Comment 6

11 years ago
> 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
(Reporter)

Comment 7

11 years ago
[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
(Reporter)

Updated

11 years ago
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.