345749 - Page Setup window does not remember its last position when reopened

Status: RESOLVED FIXED

(Core :: Printing: Output, defect)

Description

[Mats Palmgren (inactive)]

Page Setup window does not remember its last position when reopened.

Also, the following assertion is occurs when opening the window:
###!!! ASSERTION: getElementById(""), fix caller?: '!aId.IsEmpty()', file nsXULDocument.cpp, line 1595

STEPS TO REPRODUCE
1. File -> Page Setup

BUILDS AND PLATFORMS TESTED
Bug occurs in SeaMonkey trunk debug build on Linux
Bug occurs in Firefox 2.0b1 on Linux

Comment 1

[Mats Palmgren (inactive)]

Attachment: stack

This is the stack when the assertion occurs, the window is trying to
persist screenX/Y using id="" ...

Comment 2

[Mats Palmgren (inactive)]

Attachment: Patch rev. 1

This makes the window appear in its last position and does not trigger
the assertion.

Comment 3

[[e-mail]]

Comment on attachment 230463 [details] [diff] [review]
Patch rev. 1

For some reason (I searched my disk and bugzilla and found nothing relevant) my local tree already has this file patched, but with an id of printPageSetupDialog, which I suggest is slightly more accurate ;-)

Comment 4

[Mats Palmgren (inactive)]

Attachment: Patch rev. 2

printPageSetupDialog it is...

Comment 5

[Mats Palmgren (inactive)]

Comment on attachment 230584 [details] [diff] [review]
Patch rev. 2

Checked in to trunk 2006-07-25 08:50 PDT.