Bug 120456 – [AX][ATK] Added support for sort and help attributes.

• Component: Accessibility
• Status: RESOLVED FIXED

[Krzysztof Czech]

Added support for aria-help and aria-sort attributes.
Sharing aria-sort.html specific mac test with other ports.

[Radar WebKit Bug Importer]

<rdar://problem/14866030>

[Krzysztof Czech]

Created attachment 209960 [details]
Patch

[WebKit Commit Bot]

Attachment 209960 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/accessibility/aria-sort-expected.txt', u'LayoutTests/accessibility/aria-sort.html', u'LayoutTests/platform/efl-wk2/TestExpectations', u'LayoutTests/platform/efl-wk2/accessibility/image-link-expected.txt', u'LayoutTests/platform/efl-wk2/accessibility/image-map2-expected.txt', u'LayoutTests/platform/efl-wk2/accessibility/table-cell-spans-expected.txt', u'LayoutTests/platform/efl-wk2/accessibility/table-cells-expected.txt', u'LayoutTests/platform/gtk/accessibility/image-link-expected.txt', u'LayoutTests/platform/gtk/accessibility/image-map2-expected.txt', u'LayoutTests/platform/gtk/accessibility/table-cell-spans-expected.txt', u'LayoutTests/platform/gtk/accessibility/table-cells-expected.txt', u'LayoutTests/platform/mac/accessibility/aria-sort-expected.txt', u'LayoutTests/platform/mac/accessibility/aria-sort.html', u'Source/WebCore/ChangeLog', u'Source/WebCore/accessibility/atk/WebKitAccessibleWrapperAtk.cpp', u'Tools/ChangeLog', u'Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp', u'Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp']" exit_code: 1
Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:55:  An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Total errors found: 1 in 16 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Krzysztof Czech]

Created attachment 209961 [details]
Patch

[chris fleizach]

Comment on attachment 209961 [details]
Patch

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

> Source/WebCore/ChangeLog:11
> +              accessibility/aria-help.html

the change log says aria-help is added, but i don't see it in this patch

> Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp:54
> +    return attributeString == "AXSortDirection" ? "aria-sort" : String();

since it seems like you'll continue adding to this method you should probably add sort direction to the other list of if statements directly above, and then leave the return "" as the final line

> Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:58
> +    return attributeString == "AXSortDirection" ? "aria-sort" : String();

ditto with comment from DRT

[Krzysztof Czech]

Created attachment 210076 [details]
Patch

[Krzysztof Czech]

> > Source/WebCore/ChangeLog:11
> > +              accessibility/aria-help.html
> 
> the change log says aria-help is added, but i don't see it in this patch
Yes right, I did not catch this.
> 
> > Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp:54
> > +    return attributeString == "AXSortDirection" ? "aria-sort" : String();
> 
> since it seems like you'll continue adding to this method you should probably add sort direction to the other list of if statements directly above, and then leave the return "" as the final line
Yes, it seems other attributes will be added. Sounds good.
> 
> > Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:58
> > +    return attributeString == "AXSortDirection" ? "aria-sort" : String();
> 
> ditto with comment from DRT
Sounds good.

[WebKit Commit Bot]

Comment on attachment 210076 [details]
Patch

Rejecting attachment 210076 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 210076, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
 file Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp.rej
patching file Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp
Hunk #1 FAILED at 52.
Hunk #2 succeeded at 698 (offset 6 lines).
1 out of 2 hunks FAILED -- saving rejects to file Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp.rej

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Chris Fleizach']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: http://webkit-queues.appspot.com/results/1603987

[Krzysztof Czech]

Created attachment 210345 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 210345 [details]
Patch

Clearing flags on attachment: 210345

Committed r154976: <http://trac.webkit.org/changeset/154976>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.