Closed Bug 405555 Opened 17 years ago Closed 6 years ago

Navigating to another site or reloading not possible with testcase from bug 396024

Categories

(Core :: Print Preview, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: martijn.martijn, Unassigned)

References

()

Details

(Keywords: testcase)

This is (already) a follow-up from bug 396024.
If that bug is fixed, this issue remains, namely that after clicking the "Mozilla should not crash when clicking on this button" button, you can't reload the page, for example.
Also navigating to another site is not possible.

(you need to download the testcase to your computer, because of the use of enhanced privileges)

I haven't really looked all too well of where the problem is, but I think it is this:
http://lxr.mozilla.org/seamonkey/source/layout/base/nsDocumentViewer.cpp#3917
in DocumentViewerImpl::SetIsPrinting
It's setting the mIsPrintingOrPP flag on all docshells of all frames:
http://lxr.mozilla.org/seamonkey/source/docshell/base/nsDocShell.cpp#4795 
( nsDocShell::SetIsPrinting )

So when one frame is destroyed, the top level window still thinks it is in print preview.
This code seems not suited with dealing with multiple frames and print preview in one of them.
Blocks: 396024
Just in case, (now that bug 433132 is fixed),
could you check what the current behavior is ?
Component: Printing: Output → Print Preview
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.