Bug 84044 - [GTK] Expose image links properly
Summary: [GTK] Expose image links properly
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk, LayoutTestFailure
Depends on:
Blocks: 25531 98347
  Show dependency treegraph
 
Reported: 2012-04-16 09:43 PDT by Mario Sanchez Prada
Modified: 2013-10-23 09:21 PDT (History)
6 users (show)

See Also:


Attachments
Patch proposal (12.57 KB, patch)
2013-10-23 08:01 PDT, Mario Sanchez Prada
cfleizach: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mario Sanchez Prada 2012-04-16 09:43:16 PDT
This should allow unskipping accessibility/image-link.html
Comment 1 Mario Sanchez Prada 2013-10-23 08:01:39 PDT
Created attachment 214963 [details]
Patch proposal

Turns out that all we needed to do was to properly expose the URL in DRT/WKTR, as both link and image sources are being exposed already through the AtkHyperlinkImpl/AtkHyperlink combo.

Also, turns out that image-link.html was actually not being declared as "failure", but was providing the wrong expectations.

All that is addressed in the current patch
Comment 2 chris fleizach 2013-10-23 08:48:55 PDT
Comment on attachment 214963 [details]
Patch proposal

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

does this have any affect outside of the testing infrastructure? it seems that there's only layout test and DRT changes

> Tools/ChangeLog:9
> +        attribute for the AccessibilityUIElement wrapper. For coherence

the word "attribute" is duplicated
Comment 3 Mario Sanchez Prada 2013-10-23 09:10:10 PDT
(In reply to comment #2)
> (From update of attachment 214963 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=214963&action=review
> 
> does this have any affect outside of the testing infrastructure? it seems that there's only layout test and DRT changes

No, it doesn't. At the beginning I thought it was a problem in the ATK side, but turned out that part is ok and we were missing only this bit to be able to properly test it.

Another thing is image-map1.html. We'll need some changes in the wrapper because we are not exposing properly the description for image map links when it's placed in the 'title' attribute. But that's another bug (bug 84045).

> > Tools/ChangeLog:9
> > +        attribute for the AccessibilityUIElement wrapper. For coherence
> 
> the word "attribute" is duplicated

Thanks, I'll change before landing
Comment 4 Mario Sanchez Prada 2013-10-23 09:21:56 PDT
Committed r157859: <http://trac.webkit.org/changeset/157859>