Bug 146011 - AX: Add ARIA 1.1 "cell" and "table" roles
Summary: AX: Add ARIA 1.1 "cell" and "table" roles
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on: 146887
Blocks: 136415
  Show dependency treegraph
 
Reported: 2015-06-16 02:05 PDT by James Craig
Modified: 2015-08-11 12:16 PDT (History)
14 users (show)

See Also:


Attachments
patch (19.27 KB, patch)
2015-07-10 09:35 PDT, Nan Wang
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews105 for mac-mavericks-wk2 (591.10 KB, application/zip)
2015-07-10 10:15 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews101 for mac-mavericks (551.60 KB, application/zip)
2015-07-10 10:28 PDT, Build Bot
no flags Details
patch to fix mavericks' test expectation (20.80 KB, patch)
2015-07-10 11:36 PDT, Nan Wang
no flags Details | Formatted Diff | Diff
patch to fix computedRoleString test (21.77 KB, patch)
2015-07-10 12:17 PDT, Nan Wang
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews103 for mac-mavericks (530.75 KB, application/zip)
2015-07-10 12:52 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews104 for mac-mavericks-wk2 (573.77 KB, application/zip)
2015-07-10 12:56 PDT, Build Bot
no flags Details
patch that fixed test expectation (22.49 KB, patch)
2015-07-10 13:32 PDT, Nan Wang
no flags Details | Formatted Diff | Diff
patch with windows test expectation (2.15 KB, patch)
2015-07-13 16:01 PDT, Nan Wang
cfleizach: review+
cfleizach: commit-queue+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Craig 2015-06-16 02:05:32 PDT
> The new roles can be found here:
> https://rawgit.com/w3c/aria/master/aria/aria.html#cell
> https://rawgit.com/w3c/aria/master/aria/aria.html#table

I don't think WebKit has special behavior for the interactive portion of the grid and gridcell roles, so these can probably be aliases for now. computedRoleString should be updated to use the static variants, too.
Comment 1 Radar WebKit Bug Importer 2015-06-16 02:06:46 PDT
<rdar://problem/21398946>
Comment 2 Nan Wang 2015-07-10 09:35:58 PDT
Created attachment 256581 [details]
patch
Comment 3 Build Bot 2015-07-10 10:14:55 PDT
Comment on attachment 256581 [details]
patch

Attachment 256581 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6466871326932992

New failing tests:
accessibility/roles-exposed.html
accessibility/aria-table-with-presentational-elements.html
Comment 4 Build Bot 2015-07-10 10:15:00 PDT
Created attachment 256583 [details]
Archive of layout-test-results from ews105 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 5 Build Bot 2015-07-10 10:28:29 PDT
Comment on attachment 256581 [details]
patch

Attachment 256581 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5021898299670528

New failing tests:
accessibility/roles-exposed.html
accessibility/aria-table-with-presentational-elements.html
Comment 6 Build Bot 2015-07-10 10:28:33 PDT
Created attachment 256586 [details]
Archive of layout-test-results from ews101 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 7 Nan Wang 2015-07-10 11:36:33 PDT
Created attachment 256598 [details]
patch to fix mavericks' test expectation
Comment 8 chris fleizach 2015-07-10 11:42:59 PDT
Comment on attachment 256598 [details]
patch to fix mavericks' test expectation

we only need the patch for just the new things
Comment 9 Nan Wang 2015-07-10 12:17:19 PDT
Created attachment 256600 [details]
patch to fix computedRoleString test
Comment 10 Build Bot 2015-07-10 12:52:19 PDT
Comment on attachment 256600 [details]
patch to fix computedRoleString test

Attachment 256600 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5889576182743040

New failing tests:
accessibility/aria-table-with-presentational-elements.html
Comment 11 Build Bot 2015-07-10 12:52:23 PDT
Created attachment 256602 [details]
Archive of layout-test-results from ews103 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 12 Build Bot 2015-07-10 12:56:40 PDT
Comment on attachment 256600 [details]
patch to fix computedRoleString test

Attachment 256600 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/4689070546485248

New failing tests:
accessibility/aria-table-with-presentational-elements.html
Comment 13 Build Bot 2015-07-10 12:56:44 PDT
Created attachment 256603 [details]
Archive of layout-test-results from ews104 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 14 Nan Wang 2015-07-10 13:32:46 PDT
Created attachment 256606 [details]
patch that fixed test expectation
Comment 15 WebKit Commit Bot 2015-07-10 15:05:58 PDT
Comment on attachment 256606 [details]
patch that fixed test expectation

Clearing flags on attachment: 256606

Committed r186692: <http://trac.webkit.org/changeset/186692>
Comment 16 WebKit Commit Bot 2015-07-10 15:06:04 PDT
All reviewed patches have been landed.  Closing bug.
Comment 17 Brent Fulgham 2015-07-13 09:31:35 PDT
These tests fail on Windows. Can you please add a skip for them and file a bug to implement the missing features?
Comment 18 Carlos Alberto Lopez Perez 2015-07-13 14:09:38 PDT
(In reply to comment #17)
> These tests fail on Windows. Can you please add a skip for them and file a
> bug to implement the missing features?

They fail also for GTK.
Comment 19 Nan Wang 2015-07-13 14:13:30 PDT
(In reply to comment #17)
> These tests fail on Windows. Can you please add a skip for them and file a
> bug to implement the missing features?

I'll be working on adding new expectations for not working platforms.
Comment 20 Nan Wang 2015-07-13 16:01:54 PDT
Created attachment 256740 [details]
patch with windows test expectation