Regression: AX: image labels no longer exposed to AX API in SVG test case. I believe this is a recent regression. http://www.webkit.org/blog-files/aria1.0/africa_large.svg
<rdar://problem/16520975>
Created attachment 228805 [details] patch
Comment on attachment 228805 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=228805&action=review This patch looks straightforward to me. > Source/WebCore/accessibility/AccessibilityNodeObject.cpp:1849 > static String accessibleNameForNode(Node* node) Nit: We should change the parameter type from Node* to Node& since this function assumes node is a non-null pointer. Otherwise, we should check/assert that node is non-null. > LayoutTests/accessibility/svg-labelledby.html:6 > +<body id="body"> The HTML attribute id is unnecessary as we never reference this element by id in this document. Moreover, it's sufficient it use document.body to obtain a reference to the HTML Body element instead of assigning an id to it. > LayoutTests/accessibility/svg-labelledby.html:13 > + </path> Nit: This closing tag has a different amount of indentation that its opening tag (line 11).
(In reply to comment #3) > (From update of attachment 228805 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=228805&action=review > > This patch looks straightforward to me. > > > Source/WebCore/accessibility/AccessibilityNodeObject.cpp:1849 > > static String accessibleNameForNode(Node* node) > > Nit: We should change the parameter type from Node* to Node& since this function assumes node is a non-null pointer. Otherwise, we should check/assert that node is non-null. > > > LayoutTests/accessibility/svg-labelledby.html:6 > > +<body id="body"> > > The HTML attribute id is unnecessary as we never reference this element by id in this document. Moreover, it's sufficient it use document.body to obtain a reference to the HTML Body element instead of assigning an id to it. > > > LayoutTests/accessibility/svg-labelledby.html:13 > > + </path> > > Nit: This closing tag has a different amount of indentation that its opening tag (line 11). Thanks, I'll take care of all of these
http://trac.webkit.org/changeset/166917