accessibility/aria-labelledby-overrides-aria-label.html failing on Windows since it was added in r89204. http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r89261%20(13941)/results.html
Test needs to be updated likely
Landed expected failure results Committed r89565: <http://trac.webkit.org/changeset/89565>
Alice, do you have an update on this
Created attachment 98573 [details] Patch
Comment on attachment 98573 [details] Patch Hm, that didn't work as I had hoped. I have moved the test in to platform/win and platform/mac, but I still need to run it on Windows to generate the appropriate expectations. I will take another look on Monday Sydney time and hopefully successfully upload the patch then.
Created attachment 98839 [details] Patch
Comment on attachment 98573 [details] Patch Interim patch; will update Windows test and expectations from Windows machine.
Comment on attachment 98839 [details] Patch Seems like the Windows results need to be updated, not just deleted. Maybe that's what you meant by "interim"? If so, you probably shouldn't have marked the patch r?. It's really unfortunate that we have to duplicate the test for the two platforms. Maybe we should just dump the attributes instead of using shouldBe? Then each platform could have its own expected results using the same test and you wouldn't have any FAIL text.
(In reply to comment #8) > (From update of attachment 98839 [details]) > Seems like the Windows results need to be updated, not just deleted. Maybe that's what you meant by "interim"? If so, you probably shouldn't have marked the patch r?. Yes; sorry, I should have passed --no-review and forgot. But yes, that is what I meant by "interim" (and I did add that I intended to update the test and expectations from my Windows machine). > It's really unfortunate that we have to duplicate the test for the two platforms. Maybe we should just dump the attributes instead of using shouldBe? Then each platform could have its own expected results using the same test and you wouldn't have any FAIL text. Ah, that would be a better solution, indeed. I will do so and update the patch.
Created attachment 99033 [details] Patch
Created attachment 99201 [details] Patch
Created attachment 99205 [details] Patch
Created attachment 99209 [details] Patch
Comment on attachment 99209 [details] Patch Clearing flags on attachment: 99209 Committed r90226: <http://trac.webkit.org/changeset/90226>
All reviewed patches have been landed. Closing bug.