Summary: | AX: Fix accessibility/mac/search-with-frames.html test | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nan Wang <n_wang> | ||||||||
Component: | Accessibility | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aboxhall, apinheiro, cfleizach, commit-queue, dmazzoni, jcraig, jdiggs, mario, n_wang, samuel_white, thorton, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
Nan Wang
2015-08-21 10:40:56 PDT
Created attachment 259632 [details]
patch
Comment on attachment 259632 [details] patch Clearing flags on attachment: 259632 Committed r188770: <http://trac.webkit.org/changeset/188770> All reviewed patches have been landed. Closing bug. The test is still failing, although not as frequently as before. It should be marked as flaky again. I remarked it in http://trac.webkit.org/changeset/188800 (In reply to comment #4) > The test is still failing, although not as frequently as before. It should > be marked as flaky again. I'll investigate. Created attachment 259795 [details]
patch
Comment on attachment 259795 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=259795&action=review > LayoutTests/accessibility/mac/search-with-frames.html:42 > + if(ifrm.name == "frame1a") need a space after if and before ( i would also rename ifrm -> "iframe" just for clarity > LayoutTests/accessibility/mac/search-with-frames.html:44 > + else if(ifrm.name == "frame2a") ditto about spacing > LayoutTests/accessibility/mac/search-with-frames.html:49 > + // call runTest() when all frames have been loaded make this a full sentence "Call runTest... have been loaded." Created attachment 259799 [details]
patch
Comment on attachment 259799 [details] patch Clearing flags on attachment: 259799 Committed r188901: <http://trac.webkit.org/changeset/188901> All reviewed patches have been landed. Closing bug. |