98348 2012-10-04 00:18:00 -0700 [GTK] Reevaluate the platform-specific nature of some accessibility tests 2017-03-11 11:03:19 -0800 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified NEW Gtk, LayoutTestFailure P2 Normal --- 118968 141074 118966 118967 118969 123893 155353 98347 1 zan webkit-unassigned bugs-noreply clopez fred.wang jdiggs mario oldest_to_newest 734422 0 zan 2012-10-04 00:18:00 -0700 The following can currently be found in TestExpectations file: # Accessibility tests that should never be run for GTK due to be # written in a platform-specific way. They should either be moved to # the right platform or re-written in a more platform-agnostic way. accessibility/aria-list-and-listitem.html [ WontFix ] accessibility/aria-option-role.html [ WontFix ] accessibility/internal-link-anchors2.html [ WontFix ] accessibility/radio-button-group-members.html [ WontFix ] accessibility/table-attributes.html [ WontFix ] accessibility/table-detection.html [ WontFix ] accessibility/table-one-cell.html [ WontFix ] accessibility/table-sections.html [ WontFix ] accessibility/table-with-aria-role.html [ WontFix ] accessibility/table-with-rules.html [ WontFix ] accessibility/plugin.html [ WontFix ] I think we should reevaluate this classification, see if any test could be rewritten in platform-independent way and otherwise either remove these tests under proper platform-specific directories or just keep these expectations as they are. 910645 1 mario 2013-07-22 06:08:06 -0700 (In reply to comment #0) > The following can currently be found in TestExpectations file: > > # Accessibility tests that should never be run for GTK due to be > # written in a platform-specific way. They should either be moved to > # the right platform or re-written in a more platform-agnostic way. > accessibility/internal-link-anchors2.html [ WontFix ] > accessibility/radio-button-group-members.html [ WontFix ] > accessibility/table-attributes.html [ WontFix ] > accessibility/table-detection.html [ WontFix ] > accessibility/table-one-cell.html [ WontFix ] > accessibility/table-sections.html [ WontFix ] > accessibility/table-with-aria-role.html [ WontFix ] > accessibility/table-with-rules.html [ WontFix ] > accessibility/plugin.html [ WontFix ] > > I think we should reevaluate this classification, see if any test could be > rewritten in platform-independent way and otherwise either remove these tests > under proper platform-specific directories or just keep these expectations as > they are. I agree. Actually, all those tests[*] could be fixed by implementing the corresponding attributesOf*() function in AccessibilityUIElement() for DRT/WKTR, since those functions are currently unimplemented. More specifically, this is what needs to be implemented: * table-attributes.html, table-sections.html: - attributesOfColumnHeaders() - attributesOfRowHeaders() - attributesOfColumns() - attributesOfRows() - attributesOfHeader() - attributesOfVisibleCell() * plugin.html, table-with-aria-role.html: - attributesOfChildren() * internal-link-anchors2.html, radio-button-group-members.html: - attributesOfLinkedUIElements * table-detection.html, table-one-cell.html, table-with-rules.html: - allAtributes() Additionally, implementing all those functions (specially allAttributes() and attributesOfChildren()) will help us run properly other tests as well, so it looks indeed like something interesting to fix. I'll be filing bugs for all of them soon, and hopefully working on them too. [*] With the exception of aria-list-and-listitem.html and accessibility/aria-option-role.html, which are no longer in the TestExpectations file. 971607 2 clopez 2014-01-24 08:21:11 -0800 accessibility/internal-link-anchors2.html is passing correctly now on wk1/wk2 gtk+. I checked the test and I don't see anything platform specific. I will be removing it from the list. 1173641 3 fred.wang 2016-03-11 01:20:16 -0800 (In reply to comment #2) > accessibility/internal-link-anchors2.html is passing correctly now on > wk1/wk2 gtk+. I checked the test and I don't see anything platform specific. > I will be removing it from the list. This one is still marked [ WontFix Missing ] so it always generates a text expectation when you run accessibility tests. @Joanie: can you check whether the result is correct? If so, I think we can just add the missing expectation file and remove this test from TestExpectation. 1173667 4 fred.wang 2016-03-11 02:07:26 -0800 I added accessibility/content-editable-as-textarea.html to that list. http://trac.webkit.org/changeset/198003 1174469 5 fred.wang 2016-03-14 08:31:02 -0700 http://trac.webkit.org/changeset/198130