Bug 123884 - [ATK] accessibility/tabindex-elements-are-accessible.html is failing
Summary: [ATK] accessibility/tabindex-elements-are-accessible.html is failing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
: 123738 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-11-06 02:07 PST by Krzysztof Czech
Modified: 2013-11-06 03:29 PST (History)
14 users (show)

See Also:


Attachments
patch (2.47 KB, patch)
2013-11-06 02:19 PST, Krzysztof Czech
no flags Details | Formatted Diff | Diff
patch (3.24 KB, patch)
2013-11-06 02:27 PST, Krzysztof Czech
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Krzysztof Czech 2013-11-06 02:07:32 PST
I think we should generate a new expectation for GTK, EFL. It seems ATK represents paragraph's role as a ATK_ROLE_PARAGRAPH => AXParagraph. On MAC it's a AXGroup.
Comment 1 Krzysztof Czech 2013-11-06 02:19:48 PST
Created attachment 216157 [details]
patch
Comment 2 Krzysztof Czech 2013-11-06 02:27:52 PST
Created attachment 216158 [details]
patch
Comment 3 Krzysztof Czech 2013-11-06 02:30:10 PST
I'm attaching a buildbot log:
--- /mnt/buildbot/efl-linux-slave-2/efl-linux-64-release-wk2/build/layout-test-results/accessibility/tabindex-elements-are-accessible-expected.txt
+++ /mnt/buildbot/efl-linux-slave-2/efl-linux-64-release-wk2/build/layout-test-results/accessibility/tabindex-elements-are-accessible-actual.txt
@@ -19,7 +19,7 @@


 Item 4: Is focusable: true
-Item 4: AXRole: AXGroup
+Item 4: AXRole: AXParagraph


 PASS successfullyParsed is true
Comment 4 Artur Moryc 2013-11-06 02:31:24 PST
*** Bug 123738 has been marked as a duplicate of this bug. ***
Comment 5 Mario Sanchez Prada 2013-11-06 03:04:00 PST
Comment on attachment 216158 [details]
patch

(In reply to comment #0)
> I think we should generate a new expectation for GTK, EFL. It seems ATK
> represents paragraph's role as a ATK_ROLE_PARAGRAPH => AXParagraph. On MAC
> it's a AXGroup.

Yes, that's right. There's another issue provoked by the patch that introduced this new layout test, but it's completely unrelated to it (I started working on that and will file a bug soon), so I'm ok with this specific fix now.
Comment 6 WebKit Commit Bot 2013-11-06 03:29:17 PST
Comment on attachment 216158 [details]
patch

Clearing flags on attachment: 216158

Committed r158741: <http://trac.webkit.org/changeset/158741>
Comment 7 WebKit Commit Bot 2013-11-06 03:29:21 PST
All reviewed patches have been landed.  Closing bug.