592189 - creation the menu button accessible asserts "wrong children repair!"

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

Let's we have toolbar containing menu button, children caching for toolbar accessible creates accessible for menu button and initialize it before it's attached to the tree. Button initialization starts popup menu generation what results in DOM attribute change event, what are handled and we operate with menuitem, menupopup accessibles, their roles what results in GetParent() call at some point for unattached accessible.

stack:
 	xul.dll!NS_DebugBreak_P(unsigned int aSeverity, const char * aStr, const char * aExpr, const char * aFile, int aLine)  Line 371 + 0xc bytes	C++
 	xul.dll!nsAccessible::GetParent()  Line 2806 + 0x32 bytes	C++
 	xul.dll!nsXULMenupopupAccessible::GetRoleInternal(unsigned int * aRole)  Line 684 + 0x12 bytes	C++
 	xul.dll!nsAccessible::GetRole(unsigned int * aRole)  Line 1312	C++
 	xul.dll!nsAccUtils::Role(nsIAccessible * aAcc)  Line 297	C++
 	xul.dll!nsXULMenuitemAccessible::GetRoleInternal(unsigned int * aRole)  Line 491 + 0x2f bytes	C++
 	xul.dll!nsAccessible::GetRole(unsigned int * aRole)  Line 1312	C++
 	xul.dll!nsAccUtils::Role(nsIAccessible * aAcc)  Line 297	C++
 	xul.dll!nsXULMenuitemAccessible::GetStateInternal(unsigned int * aState, unsigned int * aExtraState)  Line 333 + 0x21 bytes	C++
 	xul.dll!nsAccessible::GetState(unsigned int * aState, unsigned int * aExtraState)  Line 1562 + 0x19 bytes	C++
 	xul.dll!AccStateChangeEvent::AccStateChangeEvent(nsINode * aNode, unsigned int aState, int aIsExtraState)  Line 288	C++
 	xul.dll!nsDocAccessible::AttributeChangedImpl(nsIContent * aContent, int aNameSpaceID, nsIAtom * aAttribute)  Line 978 + 0x26 bytes	C++

......

 	xul.dll!nsIContent::SetAttr(int aNameSpaceID, nsIAtom * aName, const nsAString_internal & aValue, int aNotify)  Line 356	C++
 	xul.dll!nsCoreUtils::GeneratePopupTree(nsIContent * aContent, int aIsAnon)  Line 1129 + 0x2a bytes	C++
>	xul.dll!nsXULButtonAccessible::Init()  Line 120 + 0x13 bytes	C++
 	xul.dll!nsAccessibilityService::InitAccessible(nsAccessible * aAccessible, nsRoleMapEntry * aRoleMapEntry)  Line 745 + 0xd bytes	C++
 	xul.dll!nsAccessibilityService::GetOrCreateAccessible(nsINode * aNode, nsIPresShell * aPresShell, nsIWeakReference * aWeakShell, int * aIsHidden)  Line 1092 + 0x18 bytes	C++
 	xul.dll!nsAccTreeWalker::GetNextChildInternal(int aNoWalkUp)  Line 117 + 0x2d bytes	C++

Comment 1

[alexander :surkov (:asurkov)]

this one is related with bug 545465, at least they have root problem: nsAccessible::Init() is called when accessible is not in a tree.

Comment 2

[alexander :surkov (:asurkov)]

actually this one was fixed by bug 592193: we don't use GetParent, we use mParent in NativeRole().