Closed Bug 193606 Opened 22 years ago Closed 22 years ago

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

Categories

(Core :: Networking: Cookies, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: alastor, Assigned: darin.moz)

References

Details

Attachments

(1 file)

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).
.
-> mvl ;)
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 ;)
Also occurs on the "Tools - Image Manager - Manage Image Permissions" window in the Mozilla 1.3b build.
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?
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.
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.
Confirmed on 1.3b/win2k. The window grows by around 10px each time - different from your 1px on linux :)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch remove sizeToContent (deleted) — Splinter Review
According to dwitte, removing sizeToContent helps. Jag says it is safe. So here is the patch.
Comment on attachment 114718 [details] [diff] [review] remove sizeToContent sr=jag
Attachment #114718 - Flags: superreview+
Attachment #114718 - Flags: review?(suresh)
Comment on attachment 114718 [details] [diff] [review] remove sizeToContent r-suresh
Attachment #114718 - Flags: review?(suresh) → review+
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 mvl@exedo.nl, fixed.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
*** Bug 198659 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: