54631 - -moz-opacity, tables, some sacrifice, and *crash*!

Status: RESOLVED WORKSFORME

(Core Graveyard :: GFX, defect, P3)

Description

[Hixie (not reading bugmail)]

Something is wrong with -moz-opacity. I'm not sure what, but for some reason
changing it on the page I'm about to attach causes it to crash.

Sorry, I don't have time to narrow it down right now.

Using Windows 2K Commercial Build 2000092808, which I believe is a branch build.

Comment 1

[Hixie (not reading bugmail)]

Attachment: Stylesheet for next test case

Comment 2

[Hixie (not reading bugmail)]

Attachment: Test case. (CRASHER!)

Comment 3

[Jeffrey Baker]

Doesn't crash on Linux 2000-09-28-08 M18.

Comment 4

[Kurt Weinschenker]

Doesn't seem to crash under Mac OS 9.0.4, either.
Tested with 2000092804-M18 build.

Comment 5

[Hixie (not reading bugmail)]

Taking QA.

Comment 6

[Peter Lubczynski]

The testcase does indeed crash on NT 4. Here is the stack trace:

nsBlender::Do32Blend(unsigned char 226, int 19, int 16, unsigned char * 
0x0c720000, unsigned char * 0x0c6e0000, unsigned char * 0x0c730000, int 32, int 
1024, nsBlendQuality nsHighQual, unsigned int 4278190335, unsigned int 
4294901760, nsPixelFormat & {...}) line 317 + 3 bytes
nsBlender::Blend(unsigned char * 0x0c720000, int 32, int 16, unsigned char * 
0x0c6e0000, int 1024, int 16, unsigned char * 0x0c730000, int 32, int 16, int 
19, int 89, nsPixelFormat & {...}, unsigned int 4278190335, unsigned int 
4294901760) line 224
nsBlender::Blend(nsBlender * const 0x0c623640, int 0, int 0, int 4, int 19, void 
* 0x0c623480, void * 0x0c623590, int 0, int 0, float 0.900000, void * 
0x0c623370, unsigned int 4278190335, unsigned int 4294901760) line 171 + 102 
bytes
nsBlender::Blend(nsBlender * const 0x0c623640, int 0, int 0, int 4, int 19, 
nsIRenderingContext * 0x0c623ee0, nsIRenderingContext * 0x0c623250, int 0, int 
0, float 0.900000, nsIRenderingContext * 0x0c623d60, unsigned int 4278190335, 
unsigned int 4294901760) line 202
nsViewManager2::RenderDisplayListElement(DisplayListElement2 * 0x0c627100, 
nsIRenderingContext & {...}) line 911
nsViewManager2::RenderViews(nsIView * 0x0c4b58e0, nsIRenderingContext & {...}, 
const nsRect & {...}, int & 0) line 806
nsViewManager2::Refresh(nsIView * 0x0c4b58e0, nsIRenderingContext * 0x0c6272d0, 
const nsRect * 0x0012f6f8, unsigned int 1) line 686
nsViewManager2::DispatchEvent(nsViewManager2 * const 0x0c1ebee0, nsGUIEvent * 
0x0012f838, nsEventStatus * 0x0012f73c) line 1352
HandleEvent(nsGUIEvent * 0x0012f838) line 68
nsWindow::DispatchEvent(nsWindow * const 0x0c4b57a4, nsGUIEvent * 0x0012f838, 
nsEventStatus & nsEventStatus_eIgnore) line 681 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012f838, nsEventStatus & 
nsEventStatus_eIgnore) line 707
nsWindow::OnPaint() line 3698 + 28 bytes
nsWindow::ProcessMessage(unsigned int 15, unsigned int 0, long 0, long * 
0x0012fbe8) line 2805 + 17 bytes
nsWindow::WindowProc(HWND__ * 0x0032049a, unsigned int 15, unsigned int 0, long 
0) line 950 + 27 bytes

Moving over to Compositor....

Comment 7

[Kevin McCluskey (gone)]

CC'ing attinasi

Comment 8

[Kevin McCluskey (gone)]

[e-mail] can get it to crash consistently with a display depth of
true-color on WINNT. I can't get it crash when I set my display depth to true-color.

Comment 9

[Kevin McCluskey (gone)]

Marking future. For RTM we should remove the moz-opacity property from the
branch. There are a number of crashers related to this property and not enough
time to guarantee moz-opacity is solid.

bug 57307 was filed to remove the moz-opacity property for RTM.

Comment 10

[Robert Low]

This stack trace (at line 317 in Do32Blend) would seem to
indicate the same bus error crash I was having with Do24Blend.
Refer to my bug 57223 for analysis and suggested fixes.

Comment 11

[Chase Tingley]

Has anyone tried this since hyatt landed bug 78695?  I believe that stuff
included some moz-opacity work.  

Comment 12

[Jacob Kjome]

As of the fix for bug 66022, this no longer crashes.

Might want to mark this WFM?

jake

Comment 13

[Marc Attinasi]

WFM (and others) - marking it as such.