11523 - Crash on minimizing viewer

Status: VERIFIED FIXED

(Core Graveyard :: Viewer App, defect, P1)

Description

[Nisheeth Ranjan]

Run viewer.
Hit Minimize.
Crash with the following call stack:

NTDLL! 77f76274()
nsDebug::Assertion(char * 0x00466820, char * 0x00466810, char * 0x004667e4, int
1000) line 176 + 13 bytes
nsWindow::Resize(nsWindow * const 0x00f9bc14, int 180, int 0, int -212, int 32,
int 1) line 1000 + 32 bytes
nsBrowserWindow::Layout(int 0, int 0) line 1456
HandleBrowserEvent(nsGUIEvent * 0x0012fa40) line 325
nsWindow::DispatchEvent(nsWindow * const 0x00f96704, nsGUIEvent * 0x0012fa40,
nsEventStatus & nsEventStatus_eIgnore) line 498 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012fa40) line 523
nsWindow::OnResize(nsRect & {...}) line 3186 + 15 bytes
nsWindow::ProcessMessage(unsigned int 71, unsigned int 0, long 1244340, long *
0x0012fc44) line 2668 + 24 bytes
nsWindow::WindowProc(void * 0x002705e2, unsigned int 71, unsigned int 0, long
1244340) line 571 + 27 bytes

We hit an assert in nsWindow.cpp, line number 1000, that says "Negative width
passed to nsWindow::Resize".  Seems like nsBrowserWindow is passing a negative
width to nsWindow.  CCing Radha.

Comment 1

[don]

I don't see this poblem with today's build, so I'm gonna move this out to M10
and see if it shows up again.

Comment 2

[Bill Law]

This bug is still present, but it's a problem with *viewer*, not apprunner.  I'm
switching the component to "Viewer App."

Comment 3

[rickg]

The problem was that in nsBrowserWindow we would allow height (in the bounds
rect) to be negative. This is only a band-aid fix because the file is for viewer
only and not worthy of more time.

Comment 4

[leger]

Adding crash keyword

Comment 5

[John Unruh]

Verfied.