Bug 96341 - AX: svg:image not accessible
Summary: AX: svg:image not accessible
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: chris fleizach
URL:
Keywords: InRadar
: 96558 (view as bug list)
Depends on: 96599
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-10 17:35 PDT by chris fleizach
Modified: 2012-09-12 22:54 PDT (History)
2 users (show)

See Also:


Attachments
patch (3.83 KB, patch)
2012-09-12 14:09 PDT, chris fleizach
adele: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description chris fleizach 2012-09-10 17:35:37 PDT
Images within <svg> are not accessible

<svg>
<image alt="cake" x="20" y="20" width="300" height="80" xlink:href="resources/cake.png" />\
</svg>
Comment 1 chris fleizach 2012-09-12 14:06:06 PDT
*** Bug 96558 has been marked as a duplicate of this bug. ***
Comment 2 chris fleizach 2012-09-12 14:09:05 PDT
Created attachment 163690 [details]
patch
Comment 3 chris fleizach 2012-09-12 14:11:39 PDT
rdar://12272554
Comment 4 Dominic Mazzoni 2012-09-12 14:18:39 PDT
Comment on attachment 163690 [details]
patch 

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

Unofficial review.

96323 (refactor AXRenderObj -> AXNodeObj) should re-land momentarily - could you update AccessibilityNodeObject::determineAccessibilityRole as well?

> LayoutTests/accessibility/svg-image.html:23
> +        var svg = accessibilityController.focusedElement;

Maybe call this svgImage or axSvgImage to be more clear?

> LayoutTests/accessibility/svg-image.html:24
> +        shouldBe("svg.role", "'AXRole: AXImage'");

If you're going to check the exact string, this should go in platform/mac/accessibility.

How about a cross-platform test, like this?
* Add a <img> to the page also, get its role, assert that the roles of the html img and svg image are the same.
* Also, debug() the actual role and description, and add test expectations for each platform.
Comment 5 chris fleizach 2012-09-12 14:50:55 PDT
Thanks Dominic. Will do that

(In reply to comment #4)
> (From update of attachment 163690 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=163690&action=review
> 
> Unofficial review.
> 
> 96323 (refactor AXRenderObj -> AXNodeObj) should re-land momentarily - could you update AccessibilityNodeObject::determineAccessibilityRole as well?
> 
> > LayoutTests/accessibility/svg-image.html:23
> > +        var svg = accessibilityController.focusedElement;
> 
> Maybe call this svgImage or axSvgImage to be more clear?
> 
> > LayoutTests/accessibility/svg-image.html:24
> > +        shouldBe("svg.role", "'AXRole: AXImage'");
> 
> If you're going to check the exact string, this should go in platform/mac/accessibility.
> 
> How about a cross-platform test, like this?
> * Add a <img> to the page also, get its role, assert that the roles of the html img and svg image are the same.
> * Also, debug() the actual role and description, and add test expectations for each platform.
Comment 6 chris fleizach 2012-09-12 15:57:37 PDT
(In reply to comment #5)
> Thanks Dominic. Will do that
> 
> (In reply to comment #4)
> > (From update of attachment 163690 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=163690&action=review
> > 
> > Unofficial review.
> > 
> > 96323 (refactor AXRenderObj -> AXNodeObj) should re-land momentarily - could you update AccessibilityNodeObject::determineAccessibilityRole as well?
> > 

Since AXNodeObject doesn't deal with other images right now, i think it would be best to save that logic for another patch

> > > LayoutTests/accessibility/svg-image.html:23
> > > +        var svg = accessibilityController.focusedElement;
> > 
> > Maybe call this svgImage or axSvgImage to be more clear?
> > 
> > > LayoutTests/accessibility/svg-image.html:24
> > > +        shouldBe("svg.role", "'AXRole: AXImage'");
> > 
> > If you're going to check the exact string, this should go in platform/mac/accessibility.
> > 
> > How about a cross-platform test, like this?
> > * Add a <img> to the page also, get its role, assert that the roles of the html img and svg image are the same.
> > * Also, debug() the actual role and description, and add test expectations for each platform.
Comment 7 chris fleizach 2012-09-12 15:57:52 PDT
http://trac.webkit.org/changeset/128376
Comment 8 Csaba Osztrogonác 2012-09-12 21:37:41 PDT
(In reply to comment #7)
> http://trac.webkit.org/changeset/128376

It broke the !ENABLE(SVG) builds. Could you check it, please?
https://bugs.webkit.org/show_bug.cgi?id=96599
Comment 9 chris fleizach 2012-09-12 21:46:54 PDT
(In reply to comment #8)
> (In reply to comment #7)
> > http://trac.webkit.org/changeset/128376
> 
> It broke the !ENABLE(SVG) builds. Could you check it, please?
> https://bugs.webkit.org/show_bug.cgi?id=96599

Will fix right now
Comment 10 chris fleizach 2012-09-12 22:54:18 PDT
(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > http://trac.webkit.org/changeset/128376
> > 
> > It broke the !ENABLE(SVG) builds. Could you check it, please?
> > https://bugs.webkit.org/show_bug.cgi?id=96599
> 
> Will fix right now

Build fix

http://trac.webkit.org/changeset/128405