119317 2013-07-30 21:31:00 -0700 Allow the DumpRenderTree runloop to be idle while waiting for a test to complete 2013-08-12 20:35:20 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 simon.fraser simon.fraser ap benjamin commit-queue mitz simon.fraser oldest_to_newest 913177 0 simon.fraser 2013-07-30 21:31:00 -0700 Allow the DumpRenderTree runloop to be idle while waiting for a test to complete 913179 1 207807 simon.fraser 2013-07-30 21:33:52 -0700 Created attachment 207807 Patch 915307 2 207807 commit-queue 2013-08-07 19:12:05 -0700 Comment on attachment 207807 Patch Clearing flags on attachment: 207807 Committed r153811: <http://trac.webkit.org/changeset/153811> 915308 3 commit-queue 2013-08-07 19:12:08 -0700 All reviewed patches have been landed. Closing bug. 916177 4 benjamin 2013-08-11 22:20:17 -0700 This is really cool! Why did you need the timeout? Wouldn't CFRunLoopStop(CFRunLoopGetMain()); exit the loop? 916512 5 simon.fraser 2013-08-12 20:35:20 -0700 I didn't really need to timeout, but it was a failsafe in case we relied on some other side effect of the old behavior. 207807 2013-07-30 21:33:52 -0700 2013-08-07 19:12:05 -0700 Patch bug-119317-20130730213352.patch text/plain 1947 simon.fraser U3VidmVyc2lvbiBSZXZpc2lvbjogMTUzNDU1CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggZjk4MWEzMDc5MzljN2FlYmM5YmUyZmQ1NTllZDY0NDY1 YjVjYjQ1NC4uNzAyODdlNjFiNDY1MzA4OTY2YjRiYjRiMjY0YWFiNTI5NTFiNTEyNCAxMDA2NDQK LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIy IEBACisyMDEzLTA3LTMwICBTaW1vbiBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgor CisgICAgICAgIEFsbG93IHRoZSBEdW1wUmVuZGVyVHJlZSBydW5sb29wIHRvIGJlIGlkbGUgd2hp bGUgd2FpdGluZyBmb3IgYSB0ZXN0IHRvIGNvbXBsZXRlCisgICAgICAgIGh0dHBzOi8vYnVncy53 ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTkzMTcKKworICAgICAgICBSZXZpZXdlZCBieSBO T0JPRFkgKE9PUFMhKS4KKyAgICAgICAgCisgICAgICAgIFRoZSBjaGFuZ2UgaW4gcjU1MjAwIG1h ZGUgRHVtcFJlbmRlclRyZWUgc3BpbiBhIGJ1c3ktbG9vcCB3aGlsZSB3YWl0aW5nIGZvcgorICAg ICAgICB0aGUgdGVzdCB0byBjb21wbGV0ZSwgd2hpY2ggaXMgbm90IHBvd2VyIGZyaWVuZGx5LiBB bHRob3VnaCBEUlQgaW5zdGFuY2VzCisgICAgICAgIGFyZSBnZW5lcmFsbHkgYnVzeSBhbnl3YXks IG5vdCBzaXR0aW5nIGluIGEgYnVzeSBsb29wIHNob3VsZCBmcmVlIHVwIHNvbWUgQ1BVCisgICAg ICAgIGN5Y2xlcyBmb3Igb3RoZXIgaW5zdGFuY2VzIG9mIERSVC4KKyAgICAgICAgCisgICAgICAg IEVkaXRpbmcgdGVzdHMgY29udGludWUgdG8gcGFzcy4gQWxsIEphdmEgdGVzdHMgYXJlIHNraXBw ZWQgYW5kIHdlcmUgbm90CisgICAgICAgIHRlc3RlZCwgYnV0IHdoZW4gcnVuIGZhaWwgc2ltaWxh cmx5IHdpdGggYW5kIHdpdGhvdXQgdGhlIHBhdGNoLgorCisgICAgICAgICogRHVtcFJlbmRlclRy ZWUvbWFjL0R1bXBSZW5kZXJUcmVlLm1tOgorICAgICAgICAoZHVtcCk6CisgICAgICAgIChydW5U ZXN0KToKKwogMjAxMy0wNy0yOSAgQ29tbWl0IFF1ZXVlICA8Y29tbWl0LXF1ZXVlQHdlYmtpdC5v cmc+CiAKICAgICAgICAgVW5yZXZpZXdlZCwgcm9sbGluZyBvdXQgcjE1Mjg2MC4KZGlmZiAtLWdp dCBhL1Rvb2xzL0R1bXBSZW5kZXJUcmVlL21hYy9EdW1wUmVuZGVyVHJlZS5tbSBiL1Rvb2xzL0R1 bXBSZW5kZXJUcmVlL21hYy9EdW1wUmVuZGVyVHJlZS5tbQppbmRleCAyZjMwMWM4YTIyYWY0Y2Ey Njk3ODkyYmIxOWM1NDRlYWZhMTJhM2ZiLi5jOWE1MjhhNGM3ZWU4MDZhNTVkOTA3MmYyYWExMzZh ZjQzMGY2NDJkIDEwMDY0NAotLS0gYS9Ub29scy9EdW1wUmVuZGVyVHJlZS9tYWMvRHVtcFJlbmRl clRyZWUubW0KKysrIGIvVG9vbHMvRHVtcFJlbmRlclRyZWUvbWFjL0R1bXBSZW5kZXJUcmVlLm1t CkBAIC0xMjUwLDYgKzEyNTAsNyBAQCB2b2lkIGR1bXAoKQogICAgIGZmbHVzaChzdGRlcnIpOwog CiAgICAgZG9uZSA9IFlFUzsKKyAgICBDRlJ1bkxvb3BTdG9wKENGUnVuTG9vcEdldE1haW4oKSk7 CiB9CiAKIHN0YXRpYyBib29sIHNob3VsZExvZ0ZyYW1lTG9hZERlbGVnYXRlcyhjb25zdCBjaGFy KiBwYXRoT3JVUkwpCkBAIC0xNDA1LDcgKzE0MDYsNyBAQCBzdGF0aWMgdm9pZCBydW5UZXN0KGNv bnN0IHN0cmluZyYgaW5wdXRMaW5lKQogCiAgICAgd2hpbGUgKCFkb25lKSB7CiAgICAgICAgIHBv b2wgPSBbW05TQXV0b3JlbGVhc2VQb29sIGFsbG9jXSBpbml0XTsKLSAgICAgICAgW1tOU1J1bkxv b3AgY3VycmVudFJ1bkxvb3BdIHJ1bk1vZGU6TlNEZWZhdWx0UnVuTG9vcE1vZGUgYmVmb3JlRGF0 ZTpbTlNEYXRlIGRpc3RhbnRQYXN0XV07IAorICAgICAgICBDRlJ1bkxvb3BSdW5Jbk1vZGUoa0NG UnVuTG9vcERlZmF1bHRNb2RlLCAxMCwgZmFsc2UpOwogICAgICAgICBbcG9vbCByZWxlYXNlXTsK ICAgICB9CiAK