Closed Bug 1552478 Opened 1 year ago Closed 1 year ago

Stop printpreview_bug396024_helper.xul creating a print preview viewer in a normal document window

Categories

(Core :: Print Preview, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: jwatt, Assigned: jwatt)

Details

Attachments

(1 file)

When smaug changed printpreview_bug396024_helper.xul in bug 487667 to use a separate window (iframe) for the static clone doc, it went unnoticed that the removal and appending of the element in test 'run4' actually inverted the order of the iframes. Since the functions printPreview() and exitprintpreview() are hardcoded to act on frames[1], that means that we subsequently in 'run5' initialize a print preview viewer in the iframe (docShell) that was being used to load the document we're testing previews of.

I would like to change nsPrintJob to have hard invariants about where the print preview doc viewer can be loaded, and loading it into an existing docShell should not be allowed. Hence the change to this test to avoid violating this invariant.

Pushed by jwatt@jwatt.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/930f0f51b681
Stop printpreview_bug396024_helper.xul creating a print preview viewer in a normal document window. r=bobowen
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.