63211 - "new folder" crashes on linux.

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[(not reading, please use [e-mail] instead)]

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.

Comment 1

[(not reading, please use [e-mail] instead)]

updating summary.  happens on local folders, too.

marking critical.

Comment 2

[Navin Gupta]

I know where the problem might be. It is related to checkins I did for another 
bug with resizing the window. 

Comment 3

[Navin Gupta]

Commenting out window.sizeToContent() fixes the crash.
window.sizeToContent() was working previously.

Comment 4

[Navin Gupta]

Attachment: Proposed fix

Comment 5

[(not reading, please use [e-mail] instead)]

who owns the real problem, that sizeToContent crashes?

this is just a dup of that bug.

Comment 6

[(not reading, please use [e-mail] instead)]

looks like #63224 was logged to cover the crash.

don't check in this patch.

Comment 7

[Navin Gupta]

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)

Comment 8

[Fabian Guisset]

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

Comment 9

[scottputterman]

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.

Comment 10

[Boris Zbarsky [:bzbarsky]]

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

Comment 11

[Sheela Ravindran]

change qa contact to myself

Comment 12

[scottputterman]

I thought we fixed this recently. If not, nominating for nsbeta1.

Comment 13

[Sheela Ravindran]

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

Comment 14

[Sheela Ravindran]

I don't see any problem creating new folders on linux buildid 2000122908.
Marking this bug as resolved. 

Comment 15

[Sheela Ravindran]

verified again on buildid: 2001010308.