[Webkit-unassigned] [Bug 122015] [ATK] Expose aria-invalid as a text attribute (not object attribute)

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=122015


Mario Sanchez Prada <mario at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #212986|0                           |1
        is obsolete|                            |
 Attachment #212986|review?                     |
               Flag|                            |
 Attachment #213096|                            |review?
               Flag|                            |




--- Comment #13 from Mario Sanchez Prada <mario at webkit.org>  2013-10-01 10:18:39 PST ---
Created an attachment (id=213096)
 --> (https://bugs.webkit.org/attachment.cgi?id=213096&action=review)
Patch proposal

(In reply to comment #12)
> [...]
> > Thus, I think I will update the patch moving that logic from the ATK
> > wrapper into AccessibilityObject::invalidStatus(). The aria-invalid
> > test should keep passing fine for the Mac as it is, although I will
> > probably add a new element with a not specified value to make sure
> > we return "true" in that case as well.
> 
> Sounds good

Done. See the patch attached. I changed the test to cover all the cases, but it should work on the Mac too.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.