754473 - Add stability test for nsDocAccessible::AttributeChangedImpl funk (see comment 2)

Status: RESOLVED WONTFIX

(Core :: Disability Access APIs, defect)

Description

[Martijn Wargers (dead)]

Attachment: testcase (uses enhanced privileges)

I'm seeing this crash with the attached testcase.

This bug was filed from the Socorro interface and is 
report bp-dd63007e-a260-4ada-bc1b-4fba82120511 .
============================================================= 
0 	xul.dll 	nsDocAccessible::AttributeChangedImpl 	accessible/src/base/nsDocAccessible.cpp:1087
1 	xul.dll 	nsDocAccessible::AttributeChanged 	accessible/src/base/nsDocAccessible.cpp:999
2 	xul.dll 	nsGenericElement::SetAttrAndNotify 	content/base/src/nsGenericElement.cpp:5366
3 	xul.dll 	nsGenericElement::SetAttr 	content/base/src/nsGenericElement.cpp:5270
4 	xul.dll 	nsGenericElement::SetAttribute 	content/base/src/nsGenericElement.cpp:2728
5 	xul.dll 	nsIDOMElement_SetAttribute 	obj-firefox/js/xpconnect/src/dom_quickstubs.cpp:4063
6 	mozjs.dll 	js::types::TypeSet::addType 	js/src/jsinferinlines.h:1158
7 	mozjs.dll 	js::Interpret 	js/src/jsinterp.cpp:2744
8 	mozjs.dll 	js::types::TypeMonitorCallSlow 	js/src/jsinfer.cpp:4832
9 	mozjs.dll 	js::RunScript 	js/src/jsinterp.cpp:479
10 	mozjs.dll 	js::Invoke 	js/src/jsinterp.cpp:571
11 	mozjs.dll 	JS_CallFunctionValue 	js/src/jsapi.cpp:5429
12 	xul.dll 	nsJSContext::CallEventHandler 	dom/base/nsJSEnvironment.cpp:1931
13 	xul.dll 	nsGlobalWindow::RunTimeout 	dom/base/nsGlobalWindow.cpp:9055
14 	xul.dll 	nsGlobalWindow::TimerCallback 	dom/base/nsGlobalWindow.cpp:9443
15 	xul.dll 	nsTimerImpl::Fire 	xpcom/threads/nsTimerImpl.cpp:508
16 	xul.dll 	nsTimerEvent::Run 	xpcom/threads/nsTimerImpl.cpp:591
17 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:656
18 	xul.dll 	mozilla::ipc::MessagePump::Run 	ipc/glue/MessagePump.cpp:145
19 	xul.dll 	MessageLoop::RunHandler 	ipc/chromium/src/base/message_loop.cc:201
20 	xul.dll 	MessageLoop::Run 	ipc/chromium/src/base/message_loop.cc:175

Comment 1

[Eitan Isaacson [:eeejay]]

I want to say this is a dup of bug 754377, but I'll let someone else make that call.

Comment 2

[alexander :surkov (:asurkov)]

bug 754377 was landed and it really fixes this one (but approach is not really best and it makes sense to add mochitests since we have a test case). So we can keep this bug but it's worth to change its summary.

Comment 3

[alexander :surkov (:asurkov)]

(In reply to alexander :surkov from comment #2)
> bug 754377 was landed and it really fixes this one (but approach is not
> really best and it makes sense to add mochitests since we have a test case).
> So we can keep this bug but it's worth to change its summary.

see bug 754377 comment #8

Comment 4

[alexander :surkov (:asurkov)]

(In reply to alexander :surkov from comment #3)
> (In reply to alexander :surkov from comment #2)
> > bug 754377 was landed and it really fixes this one (but approach is not
> > really best and it makes sense to add mochitests since we have a test case).
> > So we can keep this bug but it's worth to change its summary.
> 
> see bug 754377 comment #8

it was fixed by bug 678454. All we need to do here is to add mochitest.

Comment 5

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

(In reply to alexander :surkov from comment #4)
> (In reply to alexander :surkov from comment #3)
> > (In reply to alexander :surkov from comment #2)
> > > bug 754377 was landed and it really fixes this one (but approach is not
> > > really best and it makes sense to add mochitests since we have a test case).
> > > So we can keep this bug but it's worth to change its summary.
> > 
> > see bug 754377 comment #8
> 
> it was fixed by bug 678454. All we need to do here is to add mochitest.

What kind of mochitest?  (Good first bug?)

Comment 6

[alexander :surkov (:asurkov)]

(In reply to David Bolter [:davidb] from comment #5)

> What kind of mochitest?  (Good first bug?)

the given test case should be converted into mochitest, 'mentored' works

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Closing because no crash reported since 12 weeks.