626841 - Firefox 4.0b10pre crash in [@ nsAccessible::Role() ][@ nsAccessible::Role ]

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Marcia Knous [:marcia]]

Seen while reviewing crash stats. Crashes started showing up using 2011011800 build. http://tinyurl.com/4m548qr to the crashes which are all Windows.

Possible Pushlog regression range: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=66addc5c30ca&tochange=eb105fe0e41c

Frame 	Module 	Signature [Expand] 	Source
0 	xul.dll 	nsAccessible::Role 	accessible/src/base/nsAccessible.h:151
1 	xul.dll 	nsDocAccessible::UpdateTree 	accessible/src/base/nsDocAccessible.cpp:1898
2 	xul.dll 	nsDocAccessible::ProcessContentInserted 	accessible/src/base/nsDocAccessible.cpp:1861
3 	xul.dll 	NotificationController::ContentInsertion::Process 	accessible/src/base/NotificationController.cpp:551
4 	xul.dll 	NotificationController::WillRefresh 	accessible/src/base/NotificationController.cpp:195
5 	xul.dll 	nsRefreshDriver::Notify 	layout/base/nsRefreshDriver.cpp:254
6 	xul.dll 	nsTimerImpl::Fire 	xpcom/threads/nsTimerImpl.cpp:428
7 	xul.dll 	nsTimerEvent::Run 	xpcom/threads/nsTimerImpl.cpp:517
8 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:633
9 	xul.dll 	mozilla::ipc::MessagePump::Run 	ipc/glue/MessagePump.cpp:110
10 	xul.dll 	xul.dll@0xb2632b 	
11 	xul.dll 	MessageLoop::RunInternal 	ipc/chromium/src/base/message_loop.cc:219
12 	xul.dll 	MessageLoop::RunHandler 	ipc/chromium/src/base/message_loop.cc:202
13 	mozcrt19.dll 	_VEC_memzero 	
14 	xul.dll 	xul.dll@0x35b00f 	
15 	firefox.exe 	firefox.exe@0x1bb7 	
16 	ntdll.dll 	ntdll.dll@0x71ecc 	
17 	ntdll.dll 	ntdll.dll@0x39ea4 	
18 	firefox.exe 	firefox.exe@0x186f 	
19 	firefox.exe 	firefox.exe@0x186f

Comment 1

[Marcia Knous [:marcia]]

Regression from Bug 626841? Adding Trevor.

Comment 2

[David Bolter [:davidb] (NeedInfo me for attention)]

(In reply to comment #1)
> Regression from Bug 626841?

Marcia I think you mis-pasted?

Unfortunately I suspect:
http://hg.mozilla.org/mozilla-central/rev/eb105fe0e41c

Comment 3

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 4

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 504973 [details] [diff] [review]
patch

r=me, looks good.

Comment 5

[Mike Beltzner [:beltzner, not reading bugmail]]

Fast, speedy service. I like.

Comment 6

[alexander :surkov (:asurkov)]

landed on 2.0 - http://hg.mozilla.org/mozilla-central/rev/94271581b601