Running "firefox [url]" while in Print Preview creates new tabs & can trigger crash

NEW
Unassigned

Status

()

defect
--
critical
12 years ago
3 months ago

People

(Reporter: rforums, Unassigned)

Tracking

({crash})

Trunk
x86
Windows XP
Points:
---
Bug Flags:
blocking-firefox3 -
wanted-firefox3 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [ccbr][has draft patch])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

12 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b1) Gecko/2007110904 Firefox/3.0b1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b1) Gecko/2007110904 Firefox/3.0b1

i had two sites opened in different tabs. i put print preview option for the second tab. then i opened a saved web page on my hard disk. then i started closing the tabs in the order i opened them. after closing the saved page tab and the print preview tab firefox crashed.

Reproducible: Always

Steps to Reproduce:
1.open firefox and goto page www.yahoo.com
2.open a new tab and goto page www.google.com
3.goto File>Print preview option for the second tab
4.Now open a saved html page in your hard disk which opens in a third tab
5.now close the third tab, then the second tab which is in the print preview mode.
Actual Results:  
firefox crashes. tried it 3 times. 2 times, firefox crash reporter came. the third time the windows xp error reporting pop up came up.

Expected Results:  
firefox should have not crashed and the first tab should have come to focus.

in my firefox i have disabled the option to show the tab bar if one tab is present.
> i put print preview option for the second tab.

This is odd. The default Print Preview replaces replaces the entire
window with the PP view, there's no way to open new tabs when in
PP mode.  Have you installed any add-ons to get this feature?
(Reporter)

Comment 2

12 years ago
>  Have you installed any add-ons to get this feature?
no add-ons for that feature. when PP view is activated, yes, entire window is replaced. after this i just open a saved html page. the tabbar comes into view and shows the saved page. i then close the saved page tab, then the second one which is the print preview tab, then firefox crashes. this happens only when there is atleast 1 tab remaining after closing the print preview one.
(Reporter)

Comment 3

12 years ago
After the crash, i went to windows event viewer. in the application window, one error and one information entry was there.
The error entry said:
Faulting application firefox.exe, version 0.0.0.0, faulting module xul.dll, version 0.0.0.0, fault address 0x00259ccd.

The information entry said(source was Dr. Watson):
The application, C:\Program Files\Mozilla Firefox 3 Beta 1\firefox.exe, generated an application error The error occurred on 12/09/2007 @ 23:16:19.796 The exception generated was c0000005 at address 60609CCD (xul!JSD_IsLocked)
Ok, I'm stupid, I didn't get point 4 at first.  To clarify step 4 above:
 4a. open a Windows file explorer window
 4b. double-click a HTML file in some folder

This opens the file in a new tab in the Firefox window, and reveals the
tab bar under print preview bar.  This shouldn't happen.  I think we're
supposed to pick an existing window that isn't in print preview mode,
or if none is available, open a new window.

The bug also occurs in Firefox 2.0.0.11
Keywords: crash
What's the ID of your crash report?  (you can find it in the Crash Reports
folder -- see http://kb.mozillazine.org/Breakpad)
(Reporter)

Comment 6

12 years ago
Crash ID: bp-b204d342-a6d9-11dc-afc5-001a4bd43e5c
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-firefox3?
Version: unspecified → Trunk
Pretty odd use case, wanted-not-blocking.
Flags: wanted-firefox3+
Flags: blocking-firefox3?
Flags: blocking-firefox3-
Duplicate of this bug: 412231
Duplicate of this bug: 458902
Updating title to reflect the real bug here, as reflected in comment 4.
Summary: crash if i close individual tabs after print preview → Running "firefox [url]" while in Print Preview creates new tabs & can trigger crash
(In reply to comment #7)
> Pretty odd use case, wanted-not-blocking.

IMHO, this actually isn't that odd of a use-case.  The issue isn't the crash, it's that we allow new tabs to be opened in the foreground during print-preview.

I'm posting the steps-to-reproduce from duplicate-bug 458902, because they're much simpler than the steps given in this bug here.

(Quoting bug 458902 comment #0)
> Steps to reproduce:
>  1. Open exactly one session of Firefox, with exactly one window open.
>  2. Load about:blank (or any URL) and Print-Preview it.
>  3. Make Firefox open up a new tab, by...
>     a) running "firefox google.com" in the command line
>       or...
>     b) clicking on a link in another program
and I'd add another alternate option under step 3:
      c) double-clicking a HTML file in your OS's file-explorer
(In reply to comment #11)
> (In reply to comment #7)
> > Pretty odd use case, wanted-not-blocking.
> IMHO, this actually isn't that odd of a use-case.

I meant to add that I've been running into this bug pretty much daily -- I'll be debugging an issue in print-preview, and then I absentmindedly click a link in Thunderbird, and then my Firefox session is broken.

Comment 13

10 years ago
See also bug 409181, which is just about not being able to close print preview. Just fixing that issue should be a bit simpler than fixing the whole "allowing new tabs to open in a print preview window" issue.
Duplicate of this bug: 511856
Duplicate of this bug: 516280
Whiteboard: [ccbr]
Posted patch patch (obsolete) — Splinter Review
This seems to fix it.
Component: Tabbed Browser → General
QA Contact: tabbed.browser → general
I think the correct fix is to make sure "win" isn't in print preview mode in the OPEN_NEWTAB case, similar to how we bypass popup windows.
I tried that, that didn't work.
Let's drive this in!
Assignee: nobody → martijn.martijn
Attachment #400864 - Flags: review?(gavin.sharp)
This isn't tested, but if it works I think this is a better fix.
Assignee: martijn.martijn → gavin.sharp
Attachment #400864 - Attachment is obsolete: true
Attachment #400864 - Flags: review?(gavin.sharp)
Gavin: ping -- where are we with this one?
(In reply to comment #21)
> Gavin: ping -- where are we with this one?

Martijn, does patch work for you?
Whiteboard: [ccbr] → [ccbr][has draft patch]
Assignee: gavin.sharp → nobody
Component: General → Printing
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.