112946 2013-03-21 11:58:07 -0700 TestRunner::display driven requestAnimationFrame tests times out 2013-06-20 22:54:58 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 1 rafael.lobo rafael.lobo benjamin jamesr jturcotte kbalazs kenneth noam sergio oldest_to_newest 860530 0 rafael.lobo 2013-03-21 11:58:07 -0700 TestRunner::display driven requestAnimationFrame tests times out 860533 1 194312 rafael.lobo 2013-03-21 11:58:51 -0700 Created attachment 194312 Patch 860534 2 194312 rafael.lobo 2013-03-21 11:59:46 -0700 Comment on attachment 194312 Patch This patch is not for review yet, just to show the issue. 860543 3 rafael.lobo 2013-03-21 12:07:05 -0700 On bug 112582, I've tried to create a display-driven test, given that TestRunner::display simulates a flush on the browser and triggers rAF calls (see https://bugs.webkit.org/show_bug.cgi?id=112582#c17). It seems to make sense, but this does not work for Chromium, Qt and Nix (will test other ports). The odd thing is that if I add to the end of the js file a setTimeout which calls notifyDone, then the whole test passes. I've tried to look for a while but didn't find anything useful so far, maybe it is a problem on how we manage the work queue. There was also some discussion on whether rAF should ever be display-driven anyway, so we could also move the discussion to here. I'm assuming here that display is correct. 194312 2013-03-21 11:58:51 -0700 2013-03-21 11:59:46 -0700 Patch bug-112946-20130321155441.patch text/plain 3168 rafael.lobo U3VidmVyc2lvbiBSZXZpc2lvbjogMTQ2MzgyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggMWRlYTkyYzUzMDg4ZTFjZWEyOGI0 YmNlZGMxNDA2NGU4NjU5ZDgyZi4uZDBiZDdhNGY5ZmM3Y2I3ZTFjYTcyZDcyMjQ2ZTQ2OWJmMGY5 NzU4YiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0IEBACisyMDEzLTAzLTIxICBSYWZhZWwgQnJhbmRhbyAg PHJhZmFlbC5sb2JvQG9wZW5ib3NzYS5vcmc+CisKKyAgICAgICAgVGVzdFJ1bm5lcjo6ZGlzcGxh eSBkcml2ZW4gcmVxdWVzdEFuaW1hdGlvbkZyYW1lIHRlc3RzIHRpbWVzIG91dAorICAgICAgICBo dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTEyOTQ2CisKKyAgICAgICAg UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBmYXN0L2FuaW1hdGlvbi9k aXNwbGF5LWRyaXZlbi1yZXF1ZXN0LWFuaW1hdGlvbi1mcmFtZS1leHBlY3RlZC50eHQ6IEFkZGVk LgorICAgICAgICAqIGZhc3QvYW5pbWF0aW9uL2Rpc3BsYXktZHJpdmVuLXJlcXVlc3QtYW5pbWF0 aW9uLWZyYW1lLmh0bWw6IEFkZGVkLgorICAgICAgICAqIGZhc3QvYW5pbWF0aW9uL3NjcmlwdC10 ZXN0cy9kaXNwbGF5LWRyaXZlbi1yZXF1ZXN0LWFuaW1hdGlvbi1mcmFtZS5qczogQWRkZWQuCisK IDIwMTMtMDMtMjAgIEVyaWMgQ2FybHNvbiAgPGVyaWMuY2FybHNvbkBhcHBsZS5jb20+CiAKICAg ICAgICAgQWxsb3cgcG9ydHMgc3BlY2lmaWMgdGV4dCB0cmFjayBtZW51CmRpZmYgLS1naXQgYS9M YXlvdXRUZXN0cy9mYXN0L2FuaW1hdGlvbi9kaXNwbGF5LWRyaXZlbi1yZXF1ZXN0LWFuaW1hdGlv bi1mcmFtZS1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9mYXN0L2FuaW1hdGlvbi9kaXNwbGF5 LWRyaXZlbi1yZXF1ZXN0LWFuaW1hdGlvbi1mcmFtZS1leHBlY3RlZC50eHQKbmV3IGZpbGUgbW9k ZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMC4u YjkwMWIwNGE0NTIyZjhlNDdmOTgxZjE1NjI3YjhmZDU2NzRjMDc3MQotLS0gL2Rldi9udWxsCisr KyBiL0xheW91dFRlc3RzL2Zhc3QvYW5pbWF0aW9uL2Rpc3BsYXktZHJpdmVuLXJlcXVlc3QtYW5p bWF0aW9uLWZyYW1lLWV4cGVjdGVkLnR4dApAQCAtMCwwICsxLDkgQEAKK1Rlc3RzIGlmIGEgdGVz dFJ1bm5lci5kaXNwbGF5IGRyaXZlbiByQUYgdGVzdCB3b3VsZCBldmVyIGZpbmlzaAorCitPbiBz dWNjZXNzLCB5b3Ugd2lsbCBzZWUgYSBzZXJpZXMgb2YgIlBBU1MiIG1lc3NhZ2VzLCBmb2xsb3dl ZCBieSAiVEVTVCBDT01QTEVURSIuCisKKworUEFTUyBzdWNjZXNzZnVsbHlQYXJzZWQgaXMgdHJ1 ZQorCitURVNUIENPTVBMRVRFCisKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvYW5pbWF0 aW9uL2Rpc3BsYXktZHJpdmVuLXJlcXVlc3QtYW5pbWF0aW9uLWZyYW1lLmh0bWwgYi9MYXlvdXRU ZXN0cy9mYXN0L2FuaW1hdGlvbi9kaXNwbGF5LWRyaXZlbi1yZXF1ZXN0LWFuaW1hdGlvbi1mcmFt ZS5odG1sCm5ldyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw MDAwMDAwMDAwMDAwMDAwMDAuLjUzYjY1NGVlZGUwNmViZTUwM2NmOTAwNDM5YTE2NTIxYmQzZGY1 Y2UKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRUZXN0cy9mYXN0L2FuaW1hdGlvbi9kaXNwbGF5 LWRyaXZlbi1yZXF1ZXN0LWFuaW1hdGlvbi1mcmFtZS5odG1sCkBAIC0wLDAgKzEsOSBAQAorPCFE T0NUWVBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorPHNjcmlwdCBzcmM9Ii4uL2pzL3Jlc291cmNl cy9qcy10ZXN0LXByZS5qcyI+PC9zY3JpcHQ+Cis8L2hlYWQ+Cis8Ym9keT4KKzxzY3JpcHQgc3Jj PSJzY3JpcHQtdGVzdHMvZGlzcGxheS1kcml2ZW4tcmVxdWVzdC1hbmltYXRpb24tZnJhbWUuanMi Pjwvc2NyaXB0PgorPC9ib2R5PgorPC9odG1sPgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFz dC9hbmltYXRpb24vc2NyaXB0LXRlc3RzL2Rpc3BsYXktZHJpdmVuLXJlcXVlc3QtYW5pbWF0aW9u LWZyYW1lLmpzIGIvTGF5b3V0VGVzdHMvZmFzdC9hbmltYXRpb24vc2NyaXB0LXRlc3RzL2Rpc3Bs YXktZHJpdmVuLXJlcXVlc3QtYW5pbWF0aW9uLWZyYW1lLmpzCm5ldyBmaWxlIG1vZGUgMTAwNjQ0 CmluZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLmViZjUyMDU3 YzljZTBmY2U2ZDBhZjQ0ZWU4N2FiZDVlMjEwM2FiODAKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlv dXRUZXN0cy9mYXN0L2FuaW1hdGlvbi9zY3JpcHQtdGVzdHMvZGlzcGxheS1kcml2ZW4tcmVxdWVz dC1hbmltYXRpb24tZnJhbWUuanMKQEAgLTAsMCArMSwyNSBAQAorZGVzY3JpcHRpb24oIlRlc3Rz IGlmIGEgdGVzdFJ1bm5lci5kaXNwbGF5IGRyaXZlbiByQUYgdGVzdCB3b3VsZCBldmVyIGZpbmlz aCIpOworCit3aW5kb3cucmVxdWVzdEFuaW1hdGlvbkZyYW1lKGZ1bmN0aW9uKHRpbWUpIHsKKyAg ICB3aW5kb3cucmVxdWVzdEFuaW1hdGlvbkZyYW1lKGZ1bmN0aW9uKHRpbWUpIHsKKyAgICAgICAg d2luZG93LnJlcXVlc3RBbmltYXRpb25GcmFtZShmdW5jdGlvbih0aW1lKSB7CisgICAgICAgICAg ICB0ZXN0UGFzc2VkKCJhbGwgbmVzdGVkIHJlcXVlc3RBbmltYXRpb25GcmFtZSB3ZXJlIGhhbmRs ZWQiKTsKKyAgICAgICAgICAgIGlzU3VjY2Vzc2Z1bGx5UGFyc2VkKCk7CisKKyAgICAgICAgICAg IGlmICh3aW5kb3cudGVzdFJ1bm5lcikKKyAgICAgICAgICAgICAgICB0ZXN0UnVubmVyLm5vdGlm eURvbmUoKTsgICAgICAgICAgICAKKyAgICAgICAgfSk7CisKKyAgICAgICAgaWYgKHdpbmRvdy50 ZXN0UnVubmVyKQorICAgICAgICAgICAgdGVzdFJ1bm5lci5kaXNwbGF5KCk7CisgICAgfSk7CisK KyAgICBpZiAod2luZG93LnRlc3RSdW5uZXIpCisgICAgICAgIHRlc3RSdW5uZXIuZGlzcGxheSgp OworfSk7CisKK2lmICh3aW5kb3cudGVzdFJ1bm5lcikKKyAgICB0ZXN0UnVubmVyLmRpc3BsYXko KTsKKworaWYgKHdpbmRvdy50ZXN0UnVubmVyKQorICAgIHRlc3RSdW5uZXIud2FpdFVudGlsRG9u ZSgpOwpcIE5vIG5ld2xpbmUgYXQgZW5kIG9mIGZpbGUK