187748 - Cookie Manager: Persist screen position and size

Status: VERIFIED FIXED

(Core :: Networking: Cookies, defect)

Description

[Jeff Thieleke]

The Cookie Manager dialog (CookieViewer.xul) doesn't persist its screen position
and windows size.  This causes the dialog to always appear in the upper left
hand corner of the screen, even if the Perference dialog is positioned elsewhere.

Comment 1

[Jeff Thieleke]

Attachment: Persist the Cookie Manager's screen position and size

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 110675 [details] [diff] [review]
Persist the Cookie Manager's screen position and size

You really want to request review from a specific person...

timeless, can you check this in if it gets your r=?

Comment 3

[timeless]

done

Comment 4

[Andreas Kunz]

Jeff, Boris: didn't this fix bug 153378?

Comment 5

[Jeff Thieleke]

Yep, this patch fixes bug 153378 as well.  I've resovled bug 153378...thanks
finding it.

Comment 6

[Serge Gautherie (:sgautherie)]

The current bug should block bug 145941, as bug 153378 does.
(If it still matters.)

Comment 7

[benc]

VERIFIED: Mozilla 1.4a, Mac OS X.

If anyone can verify win + linux, please remove the relevant whiteboard comment.
If all plats verify, then please change the STATUS.

Comment 8

[benc]

VERIFIED: Mozilla 1.4b, WinXP

UPDATE: Mozilla 1.4b/Linux - this is not working for me, the manager always
appears centered in my screen.

what now?

Comment 9

[[e-mail]]

mvl: please see comment 8.

Comment 10

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

Confirmed that the cookie manager shows up centered on linux. The size is
remembered.
But this seems to be a more global thing. For example, the bookmark manager
shows up in some random place too, while it has the same persists attributes in
the code. (could not test on windows if the location is remembered there)

Is this intentional, or a linux bug?

Comment 11

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

On unix, mozilla lets the window manager handle the placements of windows. see
bug  31086 comment 12 and bug 49802 comment 16.
So this is fixed for linux too.

Comment 12

[benc]

VERIFIED! 
makes sense now that you explain it that way.