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

Status: RESOLVED WORKSFORME

(Core :: Print Preview, defect)

Description

[Martijn Wargers (dead)]

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.

Comment 1

[Serge Gautherie (:sgautherie)]

Just in case, (now that bug 433132 is fixed),
could you check what the current behavior is ?