Summary: | AX: Remove tight platform expectation coupling form roles-exposed layout test. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Samuel White <samuel_white> | ||||||||
Component: | Accessibility | Assignee: | Samuel White <samuel_white> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aboxhall, apinheiro, buildbot, bunhere, cdumez, cfleizach, commit-queue, dmazzoni, gyuyoung.kim, jcraig, jdiggs, mario, rniwa, sergio, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Mac (Intel) | ||||||||||
OS: | OS X 10.9 | ||||||||||
Attachments: |
|
Description
Samuel White
2014-06-20 14:30:05 PDT
Created attachment 233477 [details]
Patch.
You might as well pull out the role conditional check and te role description expectations too, no? It doesn’t make sense to have a PASS notice on the role if the subrole or role description are there but not being used. Just print out the actual results for each. Comment on attachment 233477 [details] Patch. Attachment 233477 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/6302044176515072 New failing tests: media/W3C/video/networkState/networkState_during_loadstart.html Created attachment 233496 [details]
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-10 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Created attachment 233756 [details]
Patch.
Removing the remaining platform specific information from the test itself and the PASS/FAIL output. Platform agnostic tests such as this one should rely on the text diffs to determine PASS/FAIL.
(In reply to comment #3) > You might as well pull out the role conditional check and te role description expectations too, no? It doesn’t make sense to have a PASS notice on the role if the subrole or role description are there but not being used. Just print out the actual results for each. Agreed. Done. Comment on attachment 233756 [details]
Patch.
gtk will also need to be updated, so please monitor the gtk bot in order to land that result
Comment on attachment 233756 [details] Patch. Clearing flags on attachment: 233756 Committed r170412: <http://trac.webkit.org/changeset/170412> All reviewed patches have been landed. Closing bug. |