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

Status: RESOLVED FIXED

(Core :: Print Preview, defect, P2)

Description

[Jonathan Watt [:jwatt]]

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.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: Bug 1552478. Stop printpreview_bug396024_helper.xul creating a print preview viewer in a normal document window. r=bobowen

Comment 2

[Pulsebot]

Pushed by [e-mail]:
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

Comment 3

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/930f0f51b681