1350 - ss:xpviewer crashes after several resizes, also messes up NT

Status: VERIFIED FIXED

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

Description

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Start xpviewer. Keep on resizing the window until xpviewer crashes (should only
take a few seconds if you do it rapidly). After that NT is screwed also and
needs to be rebooted. Symptoms include missing parts of taskbar, OS fails to
refresh background and other windows, windows placed incorrectly, buttons and
controls not working etc.

Comment 1

[Heikki Toivonen (remove -bugzilla when emailing directly)]

After reading the docs I think this should be critical bug 'cos it crashes (and
causes other problems as well). Here is my stack when the crash happened:

NTDLL! 77f76148()
nsDebug::Assertion(char * 0x002993ec, char * 0x002993e8, char * 0x002993a4, int
1736) line 95 + 13 bytes
nsRenderingContextWin::CopyOffScreenBits(nsRenderingContextWin * const
0x012e23a0, void * 0x012e2f00, int 0, int 0, const nsRect & {...}, unsigned int
1) line 1736 + 35 bytes
nsImageButton::HandleEvent(nsImageButton * const 0x0170d48c, nsGUIEvent *
0x0012f750) line 577
HandleImageButtonEvent(nsGUIEvent * 0x0012f750) line 61 + 19 bytes
nsWindow::DispatchEvent(nsWindow * const 0x0170d3f0, nsGUIEvent * 0x0012f750,
nsEventStatus & nsEventStatus_eIgnore) line 354 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012f750) line 370
nsWindow::OnPaint() line 1717 + 24 bytes
nsWindow::ProcessMessage(unsigned int 15, unsigned int 0, long 0, long *
0x0012f8c8) line 1318 + 17 bytes
nsWindow::WindowProc(void * 0x000203f2, unsigned int 15, unsigned int 0, long 0)
line 413 + 27 bytes
USER32! 77e71ab7()
USER32! 77e722dd()
NTDLL! 77f7624f()
nsToolbarItemHolder::Repaint(nsToolbarItemHolder * const 0x01237a74, int 1) line
113
nsToolbar::DoHorizontalLayout(const nsRect & {...}) line 540
nsToolbar::DoLayout(nsToolbar * const 0x0122ed5c) line 268
nsToolbar::Resize(nsToolbar * const 0x0122ecc0, unsigned int 0, unsigned int 72,
unsigned int 940, unsigned int 26, int 1) line 702
nsToolbarManager::DoLayout(nsToolbarManager * const 0x011cd1cc) line 475
nsToolbarManager::Resize(nsToolbarManager * const 0x011cd130, unsigned int 0,
unsigned int 0, unsigned int 940, unsigned int 97, int 1) line 568
nsBrowserWindow::Layout(int 940, int 609) line 2046
HandleBrowserEvent(nsGUIEvent * 0x0012fbc4) line 356
nsWindow::DispatchEvent(nsWindow * const 0x011c9130, nsGUIEvent * 0x0012fbc4,
nsEventStatus & nsEventStatus_eIgnore) line 354 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012fbc4) line 370
nsWindow::OnResize(nsRect & {...}) line 1758 + 15 bytes
nsWindow::ProcessMessage(unsigned int 71, unsigned int 0, long 1244508, long *
0x0012fcf0) line 1492 + 21 bytes
nsWindow::WindowProc(void * 0x00020306, unsigned int 71, unsigned int 0, long
1244508) line 413 + 27 bytes
USER32! 77e71ab7()
USER32! 77e72fbe()
NTDLL! 77f7624f()
USER32! 77e7288d()
USER32! 77e72918()
nsWindow::WindowProc(void * 0x00020306, unsigned int 274, unsigned int 61448,
long 26018409) line 425
USER32! 77e71ab7()
USER32! 77e71a77()
NTDLL! 77f7624f()
USER32! 77e7288d()
USER32! 77e72918()
nsWindow::WindowProc(void * 0x00020306, unsigned int 161, unsigned int 17, long
26018409) line 425
USER32! 77e71250()

Comment 2

[gem]

Reassigned to rickg. This is a stability issue that I am still seeing.

Comment 3

[rickg]

Yo Sparky. I think this may be yours (or ChrisK's) because of the contents of
the stack trace. Let me know what you find.

Comment 4

[Angus Davis]

Since this is a xpviewer problem, I'm reassigning to bill. Is this a DUP of
1463? Adding some people to the cc list.

Comment 5

[Angus Davis]

Oops, I didn't see rickg's comments saying this should be michael's :-)

Giving it back to him, but leaving bill on the cc: list. Sorry about that - my
stupidness.

Comment 6

[michaelp]

with the latest fixes checked in over the weekend, i'm not seeing this (i never
did since my changes were in before i got this bug). if others who *have* seen
this can confirm my findings, i'll mark it fixed.

Comment 7

[leger]

Per michaelp's last comment, moving to Resolved/Fixed

Comment 8

[ernie]

i'm marking this verified fixed, per comments above and
my own testing.  not seeing the crash anymore -
(i.e. xpviewer seems stable enough to sustain multiple
 rapid resizing)

Comment 9

[ernie]

forgot to mention, i verified this with the 11/23a xpviewer on NT

Comment 10

[ernie]

forgot to mention, i verified this with the 11/23a xpviewer on NT

Comment 11

[leger]

Inserting Milestone info.