105587 2012-12-20 16:05:49 -0800 accessibility/svg-remote-element.html is flaky (times out) 2012-12-20 16:59:24 -0800 1 1 1 Unclassified WebKit Accessibility 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 dmazzoni dmazzoni apinheiro cfleizach jdiggs tony webkit.review.bot oldest_to_newest 795905 0 dmazzoni 2012-12-20 16:05:49 -0800 I think there's a race condition - if the SVG image loads before this line is reached, the runAXTest function won't even be called and the test won't run: if (window.accessibilityController) { document.getElementById("svg-image").addEventListener("load", runAXTest); } I can't repro on my dev machine, but I'll bet this happens on a heavily-loaded test machine. 795938 1 tony 2012-12-20 16:30:22 -0800 Oh, maybe if the image is already in the cache? Anyway, can call runAXTest at document.onload instead? I believe onload fires after all the images are loaded. 795941 2 180439 dmazzoni 2012-12-20 16:32:29 -0800 Created attachment 180439 Patch 795944 3 180439 cfleizach 2012-12-20 16:36:02 -0800 Comment on attachment 180439 Patch thanks! 795963 4 180439 webkit.review.bot 2012-12-20 16:59:20 -0800 Comment on attachment 180439 Patch Clearing flags on attachment: 180439 Committed r138325: <http://trac.webkit.org/changeset/138325> 795964 5 webkit.review.bot 2012-12-20 16:59:24 -0800 All reviewed patches have been landed. Closing bug. 180439 2012-12-20 16:32:29 -0800 2012-12-20 16:59:20 -0800 Patch bug-105587-20121220162938.patch text/plain 2041 dmazzoni U3VidmVyc2lvbiBSZXZpc2lvbjogMTM4MjYzCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYmU5NTRhNTdiZDdhYjk3M2ZhOWU4 NDVlYzc4YzQxMDIyODUyYzY0My4uYTgwY2QzZGJjNzczNThkODU1YjM3MGFhNWQ1YjdjYWVjMjEy YTRhZCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEyLTEyLTIwICBEb21pbmljIE1henpvbmkg IDxkbWF6em9uaUBnb29nbGUuY29tPgorCisgICAgICAgIGFjY2Vzc2liaWxpdHkvc3ZnLXJlbW90 ZS1lbGVtZW50Lmh0bWwgaXMgZmxha3kgKHRpbWVzIG91dCkKKyAgICAgICAgaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEwNTU4NworCisgICAgICAgIFJldmlld2VkIGJ5 IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEF2b2lkIHJhY2UgY29uZGl0aW9uIHdoZXJlIHRo ZSB0ZXN0IHdvdWxkIG5ldmVyIHJ1biBpZiB0aGUKKyAgICAgICAgU1ZHIGltYWdlIHdhcyBhbHJl YWR5IGxvYWRlZCBiZWZvcmUgdGhlIGxvYWQgZXZlbnQgbGlzdGVuZXIKKyAgICAgICAgZ2V0cyBh ZGRlZC4KKworICAgICAgICAqIGFjY2Vzc2liaWxpdHkvc3ZnLXJlbW90ZS1lbGVtZW50Lmh0bWw6 CisKIDIwMTItMTItMjAgIFRvbnkgQ2hhbmcgIDx0b255QGNocm9taXVtLm9yZz4KIAogICAgICAg ICBbY2hyb21pdW1dIFVucmV2aWV3ZWQgZ2FyZGVuaW5nLgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVz dHMvYWNjZXNzaWJpbGl0eS9zdmctcmVtb3RlLWVsZW1lbnQuaHRtbCBiL0xheW91dFRlc3RzL2Fj Y2Vzc2liaWxpdHkvc3ZnLXJlbW90ZS1lbGVtZW50Lmh0bWwKaW5kZXggMzE2ZjEwNDU4ZTRlY2U3 MTU0MjMxYzUxNjU0NDNiMWFjYzUyMWQ0MC4uOGQ0YThlMmY5OTYwODE0MzliYTk2OTE0ZDNhYjg0 ZjJhMTI5OWVmYyAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvYWNjZXNzaWJpbGl0eS9zdmctcmVt b3RlLWVsZW1lbnQuaHRtbAorKysgYi9MYXlvdXRUZXN0cy9hY2Nlc3NpYmlsaXR5L3N2Zy1yZW1v dGUtZWxlbWVudC5odG1sCkBAIC0zLDcgKzMsNyBAQAogPGJvZHk+CiA8c2NyaXB0IHNyYz0iLi4v ZmFzdC9qcy9yZXNvdXJjZXMvanMtdGVzdC1wcmUuanMiPjwvc2NyaXB0PgogCi08aW1nIHRhYmlu ZGV4PSIwIiBpZD0ic3ZnLWltYWdlIiBzcmM9InJlc291cmNlcy9zdmctZmFjZS5zdmciIGFsdD0i aW50ZXJhY3RpdmUgU1ZHIiA+Cis8aW1nIHRhYmluZGV4PSIwIiBpZD0ic3ZnLWltYWdlIiBhbHQ9 ImludGVyYWN0aXZlIFNWRyIgPgogCiA8ZGl2IGlkPSJjb25zb2xlIj48L2Rpdj4KIDxzY3JpcHQ+ CkBAIC0yMSwxMCArMjEsNiBAQCBmdW5jdGlvbiBwYWdlWShlbGVtZW50KSB7CiAgICAgcmV0dXJu IGVsZW1lbnQuY2xpY2tQb2ludFkgLSBlbGVtZW50LmhlaWdodC8yOwogfQogCi1pZiAod2luZG93 LmFjY2Vzc2liaWxpdHlDb250cm9sbGVyKSB7Ci0gICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQo InN2Zy1pbWFnZSIpLmFkZEV2ZW50TGlzdGVuZXIoImxvYWQiLCBydW5BWFRlc3QpOwotfQotCiBm dW5jdGlvbiBydW5BWFRlc3QoKSB7ICAgIAogICAgIHZhciBjb250YWluZXIgPSBhY2Nlc3NpYmls aXR5Q29udHJvbGxlci5hY2Nlc3NpYmxlRWxlbWVudEJ5SWQoInN2Zy1pbWFnZSIpOwogCkBAIC02 NCw2ICs2MCwxMSBAQCBmdW5jdGlvbiBydW5BWFRlc3QoKSB7CiAgICAgZmluaXNoSlNUZXN0KCk7 CiB9CiAKK2lmICh3aW5kb3cuYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIpIHsKKyAgICBkb2N1bWVu dC5nZXRFbGVtZW50QnlJZCgic3ZnLWltYWdlIikuYWRkRXZlbnRMaXN0ZW5lcigibG9hZCIsIHJ1 bkFYVGVzdCk7Cit9Citkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgic3ZnLWltYWdlIikuc3JjID0g InJlc291cmNlcy9zdmctZmFjZS5zdmciOworCiA8L3NjcmlwdD4KIAogPHNjcmlwdCBzcmM9Ii4u L2Zhc3QvanMvcmVzb3VyY2VzL2pzLXRlc3QtcG9zdC5qcyI+PC9zY3JpcHQ+Cg==