Closed Bug 63211 Opened 24 years ago Closed 24 years ago

"new folder" crashes on linux.

Categories

(SeaMonkey :: MailNews: Message Display, defect)

x86
Linux
defect
Not set
critical

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: sspitzer, Assigned: naving)

References

Details

Attachments

(1 file)

my inbox is selected on nsmail-1, I hit new folder. boom. I see this on the console: Gdk-ERROR **: BadWindow (invalid Window parameter) serial 138681 error_code 3 request_code 15 minor_code 0 I'll try to get a stack trace. I'm wondering if this is related to the checkins this weekend to the new / rename folder js.
updating summary. happens on local folders, too. marking critical.
Severity: normal → critical
Summary: "new folder" on imap server crashes on linux. → "new folder" crashes on linux.
I know where the problem might be. It is related to checkins I did for another bug with resizing the window.
Commenting out window.sizeToContent() fixes the crash. window.sizeToContent() was working previously.
Attached patch Proposed fix (deleted) — Splinter Review
who owns the real problem, that sizeToContent crashes? this is just a dup of that bug.
looks like #63224 was logged to cover the crash. don't check in this patch.
window.sizeToContent() doesn't do much in case of IMAP servers (example nsmail-2). So commenting it out temporarily may be the right fix. Only when the users have the option of messages or folders-only folder that is in case of UW:IMAP servers window.sizeToContent() is required for new Folder Dialog to display the contents completely.(See bug #52564)
*** Bug 63337 has been marked as a duplicate of this bug. ***
I think it would be sufficient to mention in 63224 the changes to the code that need to be made to reproduce this problem. It sounds like this crashes on Linux and when you do new folder on Windows it creates a dialog that takes up the entire screen as soon as you choose a folder. I'm not sure it's worth keeping in to fix a bug that only helps UW Imap servers when these new bugs affect everyone else. If we think that 63224 is going to get fixed soon then we can let this wait, but otherwise, I don't think we should leave in code that breaks a commonly used feature.
*** Bug 63476 has been marked as a duplicate of this bug. ***
change qa contact to myself
QA Contact: esther → sheelar
I thought we fixed this recently. If not, nominating for nsbeta1.
Keywords: nsbeta1
*** Bug 63275 has been marked as a duplicate of this bug. ***
I don't see any problem creating new folders on linux buildid 2000122908. Marking this bug as resolved.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified again on buildid: 2001010308.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: