62987 - window.sizeToContent() does not work. If called more than once on the same window the it resizes the window to desktop size.

Status: RESOLVED INVALID

(Core :: XUL, defect, P3)

Description

[Navin Gupta]

Additional info :
  In MailNews if you try to add a new Folder window then you may have to change
the parent folder name and since the length of the parent folder name may vary
depending upon the user's choice; therefore you have to resize the window
using the size of the content. Presently to overcome this problem I use
window.resizeTo(0,0) and then use window.sizeToContent() (May indicate where the
problem is)

Comment 1

[Peter "jag" Annema]

I added some NS_WARNINGs and they seem to suggest an infinite loop from
nsWindow::Resize to nsWidget::OnResize and back, which a resizeTo(0, 0)
prevents.

Comment 2

[Peter "jag" Annema]

Currently I'm getting this error on linux, both with and without sizeTo(0, 0):
Gdk-ERROR **: BadWindow (invalid Window parameter)
  serial 3366 error_code 3 request_code 15 minor_code 0

Removing the sizeToContent lines "fixes" it. I wonder if I screwed something up
somewhere... Will get back on that.

Comment 3

[Peter "jag" Annema]

Oh, to get back on this, nope, this appears to be in the trunk. See bug 63211
and bug 63224.

Comment 4

[Peter "jag" Annema]

I suspect the fix to bug 63224 will fix this too. Marking "depends on" for now,
though it should probably be duped.

Comment 5

[Navin Gupta]

reassigning to xptoolkit. 

Comment 6

[Fabian Guisset]

All the bugs dependant on this one have been marked fixed, all the linux
crashers related to sizeToContent have been fixed too. Shall we mark this fixed?

Comment 7

[Navin Gupta]

window.sizeToContent no longer used in this part of the code. So marking this 
bug invalid