587335 - Crash [@ nsTransitionManager::WillRefresh] with transitions and reloading

Status: RESOLVED DUPLICATE of bug 592698

(Core :: CSS Parsing and Computation, defect)

Description

[Martijn Wargers (dead)]

Attachment: testcase

See testcase, which crashes current trunk build after 500ms.

http://crash-stats.mozilla.com/report/index/601fd77d-f91a-474e-a770-280e22100814
0  	xul.dll  	nsTransitionManager::WillRefresh  	 layout/style/nsTransitionManager.cpp:931
1 	xul.dll 	nsRefreshDriver::Notify 	layout/base/nsRefreshDriver.cpp:227
2 	xul.dll 	nsTimerImpl::Fire 	xpcom/threads/nsTimerImpl.cpp:428
3 	xul.dll 	nsTimerEvent::Run 	xpcom/threads/nsTimerImpl.cpp:517
4 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:547
5 	xul.dll 	mozilla::ipc::MessagePump::Run 	ipc/glue/MessagePump.cpp:118
6 	xul.dll 	xul.dll@0xa7a733 	
7 	xul.dll 	MessageLoop::RunInternal 	ipc/chromium/src/base/message_loop.cc:219
8 	xul.dll 	MessageLoop::RunHandler 	ipc/chromium/src/base/message_loop.cc:202
9 	xul.dll 	_SEH_epilog4 	
10 	xul.dll 	MessageLoop::Run 	ipc/chromium/src/base/message_loop.cc:176
11 	xul.dll 	nsBaseAppShell::Run 	widget/src/xpwidgets/nsBaseAppShell.cpp:175

Tracking bug for CSS transitions for Firefox 4 is bug 537142, btw.

Comment 1

[Martijn Wargers (dead)]

Attachment: testcase

Uh, the navigator.plugins.refresh() part isn't even necessary.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Might this be a testcase for bug 592698?  Is it fixed by the patch there?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Yes, it is.  I'll add a test for that bug based on this testcase.