189814 - Printing a XUL slider crashes browser in Classic theme [@ nsPrintEngine::ReflowPrintObject]

Status: RESOLVED WORKSFORME

(Core :: XUL, defect)

Description

[Chuck Musser]

User-Agent:       Mozilla/5.0 (X11; U; Linux i586; en-US; rv:1.3b) Gecko/20030119
Build Identifier: Mozilla/5.0 (X11; U; Linux i586; en-US; rv:1.3b) Gecko/20030119

If the Classic theme is active, printing a XUL document with a <slider> will
crash the browser. Setting the enclosed <thumb>'s CSS -moz-appearance property
to "none" prevents the crash. The crash doesn't happen at all in the Modern theme.



Reproducible: Always

Steps to Reproduce:
1. Switch to the Classic theme
2. access the attached printcrash.xul document
3. print this document.

Actual Results:  
Browser crashed.

Expected Results:  
Printed the document.

Talkback crash ID: TB16399880E

Comment 1

[Chuck Musser]

Attachment: Print this XUL document with the Classic theme will crash the browser.

Comment 2

[Andrew Schultz]

Attachment: stacktrace

from linux trunk CVS (today)

Comment 3

[Andrew Schultz]

marking NEW

Comment 4

[Roland Mainz]

WTF with 2003-03-27-08-trunk - should be fixed now...

Comment 5

[Andrew Schultz]

I still crash with linux trunk 20030330

Comment 6

[Roland Mainz]

Andrew Schultz wrote:
> I still crash with linux trunk 20030330

Printing PostScript module or Xprint module ?

Comment 7

[Andrew Schultz]

I now crash with print preview (which should be using XPrint), but not print
(XPrint and Postcript both work).

Comment 8

[Adam Hauner]

It's also crashing with 2003091510/trunk/W2K -> TB23659746M
xref: bug 219410 - print preview of about:config is crashing (with same crash
signature)

Comment 9

[Mats Palmgren (inactive)]

about:config is bug 218304

Comment 10

[Wayne Mery (:wsmwk)]

no crash currently.  receive message "We are unable to Print or Print Preview
this page."  

Comment 11

[Jesse Ruderman]

Yeah, that's because of the sketchy fix in bug 240490.

It's possible that something like this still crashes when inside an HTML document.  But Martijn Wargers has been testing print / print preview really hard, so if this bug still existed when he started I bet it's gone by now.

I'll add this testcase as crashtest, just to be sure.  I'll use reftest-print if the framework lets me.

Comment 12

[Jesse Ruderman]

Using reftest-print seems to get around the fix in bug 240490, triggering assertions like:

###!!! ASSERTION: XUL documents should never be scrollable - see above: '!isScrollable || !isXUL', file /Users/jruderman/central/layout/base/nsCSSFrameConstructor.cpp, line 2853

###!!! ASSERTION: Unexpected PopupSetFrame: 'nsIRootBox::GetRootBox(mPresShell) && nsIRootBox::GetRootBox(mPresShell)->GetPopupSetFrame() == newFrame', file /Users/jruderman/central/layout/base/nsCSSFrameConstructor.cpp, line 3903

So I'll add this as a normal crashtest.

Comment 13

[Jesse Ruderman]

Crashtest: http://hg.mozilla.org/mozilla-central/rev/006787c678a8