376514 - stop text accessible using for nsHTMLListBulletAccessible

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

Attachment: patch

Per bug 372131, we keep nsTextAccessible for MSAA uses. nsTextAccessible inherits the logic of action and states from linkable accessible. It looks nsHTMLListBulletAccessible isn't related with linkable accessibles therefore it's resonable to stop text accessible using for it.

Comment 1

[Aaron Leventhal]

Please see bug 376275.

Comment 2

[Ginn Chen]

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

it caused crashes.

stacks here:
#6  0xb6ed7e45 in nsAccessible::IsCorrectFrameType (aFrame=0x220022, 
    aAtom=0x82e9348) at nsAccessible.cpp:1310
#7  0xb6ee08aa in nsAccessible::GetBoundsRect (this=0x8cfc068, 
    aTotalBounds=@0xbf9bf25c, aBoundingFrame=0xbf9bf26c)
    at nsAccessible.cpp:1203
#8  0xb6edfdc7 in nsAccessible::GetBounds (this=0x8cfc068, x=0xbf9bf2bc, 
    y=0xbf9bf2b8, width=0xbf9bf2b4, height=0xbf9bf2b0) at nsAccessible.cpp:1278
#9  0xb6f0a54d in nsHTMLLIAccessible::GetBounds (this=0x8bb2d00, x=0xbf9bf360, 
    y=0xbf9bf35c, width=0xbf9bf358, height=0xbf9bf354)
    at nsHTMLTextAccessible.cpp:294
#10 0xb6f20cdf in getExtentsCB (aComponent=0x81681b8, aAccX=0xbf9bf428, 
    aAccY=0xbf9bf424, aAccWidth=0xbf9bf420, aAccHeight=0xbf9bf41c, 
    aCoordType=ATK_XY_SCREEN) at nsMaiInterfaceComponent.cpp:111
#11 0xb78e1936 in atk_component_get_extents (component=0x81681b8, 
    x=0xbf9bf428, y=0xbf9bf424, width=0xbf9bf420, height=0xbf9bf41c, 
    coord_type=ATK_XY_SCREEN) at atkcomponent.c:255

Comment 3

[alexander :surkov (:asurkov)]

(In reply to comment #1)
> Please see bug 376275.
> 

Looks like unrelated things.

Comment 4

[alexander :surkov (:asurkov)]

Attachment: patch2

Comment 5

[alexander :surkov (:asurkov)]

checked in