193606 - Cookie window grows (horizontally and vertically) every time on opening

Status: RESOLVED FIXED

(Core :: Networking: Cookies, defect)

Description

[Rowan "Alastor" Mulder]

User-Agent:       Opera/7.02 Bork-edition (Windows NT 5.1; U)  [en]
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3b) Gecko/20030210

When opening the cookie manager (Preferences - Privacy & Security - Cookies - 
Manage cookies) or (Tools - Cookie Manager - Manage Stored Cookies), the cookie 
manager's window will grow in horizontal and vertical direction.






Reproducible: Always

Steps to Reproduce:
1.Preferences


2.Privacy & Security


3.Cookies


4.Manage Cookies



Actual Results:  
Clicking this every time will only make it bigger until it fills the screen.


Expected Results:  
Remember the standard size (or saved one by user).

Comment 1

[Darin Fisher]

.

Comment 2

[[e-mail]]

-> mvl ;)

Comment 3

[[e-mail]]

oh, btw...

> User-Agent:       Opera/7.02 Bork-edition (Windows NT 5.1; U)  [en]

nice to see the reporter has a sense of humor ;)

Comment 4

[Rowan "Alastor" Mulder]

Also occurs on the "Tools - Image Manager - Manage Image Permissions" window in
the Mozilla 1.3b build.

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

I see it on linux 2003021305.
It is subtle, only one pixel for every time it is openend. That mean I must open
a lot for the screen to fill. Reporter, how fast is it growing for you?

Comment 6

[Michiel van Leeuwen (email: mvl+moz@)]

Ok, i found out the removing
http://lxr.mozilla.org/seamonkey/source/extensions/wallet/cookieviewer/CookieViewer.js#124
helps. I can't find out why it is there. Blame gives no hint.
But it makes me think that SizeToContent has a bug. So just removing this line
here is maybe not the right thing to do.

Comment 7

[Rowan "Alastor" Mulder]

On my Windows XP Home Edition (SP-1) it grows about 10-15 pixels every time I 
open the window, strange that it is less on Linux.

Comment 8

[[e-mail]]

Confirmed on 1.3b/win2k. The window grows by around 10px each time - different
from your 1px on linux :)

Comment 9

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: remove sizeToContent

According to dwitte, removing sizeToContent helps. Jag says it is safe. So here
is the patch.

Comment 10

[jag (Peter Annema)]

Comment on attachment 114718 [details] [diff] [review]
remove sizeToContent

sr=jag

Comment 11

[Suresh]

Comment on attachment 114718 [details] [diff] [review]
remove sizeToContent

r-suresh

Comment 12

[Robert Kaiser]

Checking in extensions/wallet/cookieviewer/CookieViewer.js;
/cvsroot/mozilla/extensions/wallet/cookieviewer/CookieViewer.js,v  <-- 
CookieViewer.js
new revision: 1.75; previous revision: 1.74
done

I checked in for [e-mail], fixed.

Comment 13

[R.K.Aa.]

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