Summary: | AX: table cells that use display:block render the table inaccessible to VoiceOver | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | chris fleizach <cfleizach> | ||||
Component: | Accessibility | Assignee: | chris fleizach <cfleizach> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aboxhall, apinheiro, ap, bfulgham, commit-queue, dmazzoni, jcraig, jdiggs, mario, samuel_white, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
chris fleizach
2015-03-24 09:31:53 PDT
Created attachment 249337 [details]
patch
Attachment 249337 [details] did not pass style-queue:
ERROR: Source/WebCore/accessibility/AccessibilityTableCell.cpp:70: Multi line control clauses should use braces. [whitespace/braces] [4]
Total errors found: 1 in 5 files
If any of these errors are false positives, please file a bug against check-webkit-style.
(In reply to comment #2) > Attachment 249337 [details] did not pass style-queue: > > > ERROR: Source/WebCore/accessibility/AccessibilityTableCell.cpp:70: Multi > line control clauses should use braces. [whitespace/braces] [4] > Total errors found: 1 in 5 files > seems like a false positive > > If any of these errors are false positives, please file a bug against > check-webkit-style. Comment on attachment 249337 [details]
patch
Thanks!
Comment on attachment 249337 [details] patch Clearing flags on attachment: 249337 Committed r181988: <http://trac.webkit.org/changeset/181988> All reviewed patches have been landed. Closing bug. This new test is broken on Windows, Efl and Gtk. Can this be fixed quickly, or should we roll out? https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=accessibility%2Ftable-cell-display-block.html @@ -1,14 +1,18 @@ +CONSOLE MESSAGE: line 29: TypeError: undefined is not an object (evaluating 'table.cellForColumnAndRow') +a1 +b1 c1 +a2 +b2 c2 +a1 +b1 c1 +a2 +b2 c2 This tests that if a table cell uses display:block, the table cell will still be accessible On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". -PASS cell1a.isEqual(cell1b) is true -PASS cell1a.role is 'AXRole: AXCell' -PASS domCell1.style.display is 'block' -PASS table2.role is 'AXRole: AXGroup' -PASS cell2.role is 'AXRole: AXStaticText' -PASS successfullyParsed is true +FAIL successfullyParsed should be true (of type boolean). Was undefined (of type undefined). We should make this a Mac only test if these other platforms don't implement cellForColAndRoe (In reply to comment #7) > This new test is broken on Windows, Efl and Gtk. Can this be fixed quickly, > or should we roll out? > > > https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard. > html#showAllRuns=true&tests=accessibility%2Ftable-cell-display-block.html > > > @@ -1,14 +1,18 @@ > +CONSOLE MESSAGE: line 29: TypeError: undefined is not an object (evaluating > 'table.cellForColumnAndRow') > +a1 > +b1 c1 > +a2 > +b2 c2 > +a1 > +b1 c1 > +a2 > +b2 c2 > This tests that if a table cell uses display:block, the table cell will > still be accessible > > On success, you will see a series of "PASS" messages, followed by "TEST > COMPLETE". > > > -PASS cell1a.isEqual(cell1b) is true > -PASS cell1a.role is 'AXRole: AXCell' > -PASS domCell1.style.display is 'block' > -PASS table2.role is 'AXRole: AXGroup' > -PASS cell2.role is 'AXRole: AXStaticText' > -PASS successfullyParsed is true > +FAIL successfullyParsed should be true (of type boolean). Was undefined (of > type undefined). Could you please skip this test on Windows (and perhaps others) and file a bug to remind us to fix this? (In reply to comment #9) > Could you please skip this test on Windows (and perhaps others) and file a > bug to remind us to fix this? Ok on it. Thanks (In reply to comment #10) > (In reply to comment #9) > > Could you please skip this test on Windows (and perhaps others) and file a > > bug to remind us to fix this? > > Ok on it. Thanks Tests skipped http://trac.webkit.org/changeset/182002 Windows https://bugs.webkit.org/show_bug.cgi?id=143081 Linux https://bugs.webkit.org/show_bug.cgi?id=143083 |