Bug 147660

Summary: AX: tree item children returned from ranged getter are different from full array of children
Product: WebKit Reporter: Doug Russell <d_russell>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, apinheiro, cfleizach, commit-queue, dmazzoni, jcraig, jdiggs, mario, samuel_white, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
patch
none
patch none

Description Doug Russell 2015-08-04 16:09:31 PDT 
Accessing the children of a tree item from -[WebAccessibilityObjectWrapper accessibilityAttributeValue:] will result in different values than asking for a range of children from -[WebAccessibilityObjectWrapper accessibilityArrayAttributeValues:index:maxCount:] because accessibilityAttributeValue: includes a check that restricts tree item children to contents only and not row children, but accessibilityArrayAttributeValues:index:maxCount: does not.
Comment 1 Doug Russell 2015-08-04 16:54:24 PDT 
Created attachment 258236 [details]
patch
Comment 2 chris fleizach 2015-08-04 17:03:35 PDT 
Comment on attachment 258236 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258236&action=review

> LayoutTests/accessibility/mac/aria-tree-item-children.html:37
> +          tree = accessibilityController.focusedElement;

i prefer using axController.accessibleElementById("tree") these days because it ends up exercising more ax code

> Source/WebCore/ChangeLog:8
> +        Test: accessibility/mac/aria-tree-item-children.html

can you add a description of what the bug was and how you fixed it in brief
Comment 3 Doug Russell 2015-08-04 18:35:25 PDT 
Created attachment 258255 [details]
patch
Comment 4 WebKit Commit Bot 2015-08-04 20:39:11 PDT 
Comment on attachment 258255 [details]
patch

Clearing flags on attachment: 258255

Committed r187935: <http://trac.webkit.org/changeset/187935>
Comment 5 WebKit Commit Bot 2015-08-04 20:39:15 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 6 James Craig 2015-08-20 00:07:39 PDT 
<rdar://problem/22097682>