79106 2012-02-21 07:04:40 -0800 Web Inspector: timeline hangs on a page with deep chain of nested events. 2012-02-24 10:50:42 -0800 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 0 yurys yurys apavlov bweinstein joepeck keishi loislo pfeldman pmuellr rik timothy yurys oldest_to_newest 560852 0 127963 yurys 2012-02-21 07:04:40 -0800 Created attachment 127963 Test page Try to run timeline on the attached page. Result: After awhile Timeline will hang. Stack overflow exception in the front-end. 560856 1 127967 yurys 2012-02-21 07:08:21 -0800 Created attachment 127967 Patch 560885 2 yurys 2012-02-21 08:14:11 -0800 Committed r108355: <http://trac.webkit.org/changeset/108355> 564446 3 timothy 2012-02-24 10:40:33 -0800 How are timers nesting anyway? I thought they had to return to the run loop before firing the next timer. I think the fact that the timeline nests them should be considered a bug. 564455 4 pfeldman 2012-02-24 10:50:42 -0800 (In reply to comment #3) > How are timers nesting anyway? I thought they had to return to the run loop before firing the next timer. I think the fact that the timeline nests them should be considered a bug. There is a timeline mode (turned on by default) that binds asynchronous events to their causes. As a result, Timer Fire falls to under the Set Timeout. In case animation is implemented as a series of such events, they all form a giant connected branch. It draws us out of stack bounds. There is now a way to disable this binding. 127963 2012-02-21 07:04:40 -0800 2012-02-21 07:04:40 -0800 Test page recursive-timeout.html text/html 122 yurys PHNjcmlwdD4KCnZhciBjb3VudGVyID0gMDsKZnVuY3Rpb24gb25UaW1lb3V0KCkKewogICAgY291 bnRlcisrOwogICAgc2V0VGltZW91dChvblRpbWVvdXQsIDApOwp9CgpvblRpbWVvdXQoKTsKCjwv c2NyaXB0Pgo= 127967 2012-02-21 07:08:21 -0800 2012-02-21 08:11:00 -0800 Patch bug-79106-20120221190816.patch text/plain 5545 yurys U3VidmVyc2lvbiBSZXZpc2lvbjogMTA4MzQ0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNzUwMWQ0YWZhMjQ1OTM1 NGI1NTRkYjJhOWRiY2ZiNTkyNmFmODJiNS4uNmJhNTFmYjUzNjNkNmNlZWRhYzBhN2MzMzE2MzNm MmVlZWMyMWNmMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDIwIEBACiAyMDEyLTAyLTIxICBZdXJ5 IFNlbWlraGF0c2t5ICA8eXVyeXNAY2hyb21pdW0ub3JnPgogCisgICAgICAgIFdlYiBJbnNwZWN0 b3I6IHRpbWVsaW5lIGhhbmdzIG9uIGEgcGFnZSB3aXRoIGRlZXAgY2hhaW4gb2YgbmVzdGVkIGV2 ZW50cy4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTc5 MTA2CisKKyAgICAgICAgVXNlIHN0YWNrIG9mIHN0YXRlcyBpbnN0ZWFkIG9mIHJlY3Vyc2l2ZSBj YWxscyB3aGVuIHRyYXZlcnNpbmcgcmVjb3JkcyB0cmVlIHRvCisgICAgICAgIGF2b2lkIHN0YWNr IG92ZXJmbG93LgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg ICAgICogaW5zcGVjdG9yL2Zyb250LWVuZC9UaW1lbGluZU92ZXJ2aWV3UGFuZS5qczoKKyAgICAg ICAgKFdlYkluc3BlY3Rvci5UaW1lbGluZU92ZXJ2aWV3UGFuZS5wcm90b3R5cGUuX2ZvckFsbFJl Y29yZHMpOgorICAgICAgICAqIGluc3BlY3Rvci9mcm9udC1lbmQvVGltZWxpbmVQYW5lbC5qczoK KyAgICAgICAgKFdlYkluc3BlY3Rvci5UaW1lbGluZVBhbmVsLnByb3RvdHlwZS5fZmlsdGVyUmVj b3Jkcyk6CisKKzIwMTItMDItMjEgIFl1cnkgU2VtaWtoYXRza3kgIDx5dXJ5c0BjaHJvbWl1bS5v cmc+CisKICAgICAgICAgW1Y4XSBXZWIgSW5zcGVjdG9yOiBzZXQgYnJlYWtwb2ludC9wYXVzZSBk b2Vzbid0IHdvcmsgd2hlbiB3b3JrZXIgaXMgaW4gYSB0aWdodCBsb29wCiAgICAgICAgIGh0dHBz Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD03OTA5NwogCmRpZmYgLS1naXQgYS9T b3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1RpbWVsaW5lT3ZlcnZpZXdQYW5lLmpz IGIvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9UaW1lbGluZU92ZXJ2aWV3UGFu ZS5qcwppbmRleCBhYThjMjc1ZDk0ZDU4OTYxMzAxMDA2M2Q2ZDE4NjE1NTZmOGY3ZjAzLi5iMzdk OGMwOTIwM2QxOTkyZDg3Mjk2Mzk4NTZkYjBlZjU5MzA4NGY2IDEwMDY0NAotLS0gYS9Tb3VyY2Uv V2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1RpbWVsaW5lT3ZlcnZpZXdQYW5lLmpzCisrKyBi L1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvVGltZWxpbmVPdmVydmlld1BhbmUu anMKQEAgLTEyNiw5ICsxMjYsMTggQEAgV2ViSW5zcGVjdG9yLlRpbWVsaW5lT3ZlcnZpZXdQYW5l LnByb3RvdHlwZSA9IHsKICAgICB7CiAgICAgICAgIGlmICghcmVjb3Jkc0FycmF5KQogICAgICAg ICAgICAgcmV0dXJuOwotICAgICAgICBmb3IgKHZhciBpID0gMDsgaSA8IHJlY29yZHNBcnJheS5s ZW5ndGg7ICsraSkgewotICAgICAgICAgICAgY2FsbGJhY2socmVjb3Jkc0FycmF5W2ldKTsKLSAg ICAgICAgICAgIHRoaXMuX2ZvckFsbFJlY29yZHMocmVjb3Jkc0FycmF5W2ldLmNoaWxkcmVuLCBj YWxsYmFjayk7CisgICAgICAgIHZhciBzdGFjayA9IFt7YXJyYXk6IHJlY29yZHNBcnJheSwgaW5k ZXg6IDB9XTsKKyAgICAgICAgd2hpbGUgKHN0YWNrLmxlbmd0aCkgeworICAgICAgICAgICAgdmFy IGVudHJ5ID0gc3RhY2tbc3RhY2subGVuZ3RoIC0gMV07CisgICAgICAgICAgICB2YXIgcmVjb3Jk cyA9IGVudHJ5LmFycmF5OworICAgICAgICAgICAgaWYgKGVudHJ5LmluZGV4IDwgcmVjb3Jkcy5s ZW5ndGgpIHsKKyAgICAgICAgICAgICAgICAgdmFyIHJlY29yZCA9IHJlY29yZHNbZW50cnkuaW5k ZXhdOworICAgICAgICAgICAgICAgICBjYWxsYmFjayhyZWNvcmQpOworICAgICAgICAgICAgICAg ICBpZiAocmVjb3JkLmNoaWxkcmVuKQorICAgICAgICAgICAgICAgICAgICAgc3RhY2sucHVzaCh7 YXJyYXk6IHJlY29yZC5jaGlsZHJlbiwgaW5kZXg6IDB9KTsKKyAgICAgICAgICAgICAgICAgKytl bnRyeS5pbmRleDsKKyAgICAgICAgICAgIH0gZWxzZQorICAgICAgICAgICAgICAgIHN0YWNrLnBv cCgpOwogICAgICAgICB9CiAgICAgfSwKIApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaW5z cGVjdG9yL2Zyb250LWVuZC9UaW1lbGluZVBhbmVsLmpzIGIvU291cmNlL1dlYkNvcmUvaW5zcGVj dG9yL2Zyb250LWVuZC9UaW1lbGluZVBhbmVsLmpzCmluZGV4IDNkYWIwZmEzMjZiY2ExNDIwYWU1 YzJkZTIwN2M4MjE0OWRmNDgxN2UuLjVkMGI3ZjdlNDRlYWVkN2M5YmI0NWMwN2Y3ZTkyODE5MWFl NTA3MWUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvVGlt ZWxpbmVQYW5lbC5qcworKysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1Rp bWVsaW5lUGFuZWwuanMKQEAgLTY3MiwzNCArNjcyLDQzIEBAIFdlYkluc3BlY3Rvci5UaW1lbGlu ZVBhbmVsLnByb3RvdHlwZSA9IHsKICAgICAgICAgdGhpcy5fY2FsY3VsYXRvci5jYWxjdWxhdGVX aW5kb3coKTsKICAgICB9LAogCi0gICAgLyoqCi0gICAgICogQHBhcmFtIHtib29sZWFuPX0gcGFy ZW50SXNDb2xsYXBzZWQKLSAgICAgKi8KLSAgICBfYWRkVG9SZWNvcmRzV2luZG93OiBmdW5jdGlv bihyZWNvcmQsIHJlY29yZHNXaW5kb3csIHBhcmVudElzQ29sbGFwc2VkKQotICAgIHsKLSAgICAg ICAgaWYgKCF0aGlzLl9zaG93U2hvcnRFdmVudHMgJiYgIXJlY29yZC5pc0xvbmcoKSkKLSAgICAg ICAgICAgIHJldHVybjsKLSAgICAgICAgdmFyIHBlcmNlbnRhZ2VzID0gdGhpcy5fY2FsY3VsYXRv ci5jb21wdXRlQmFyR3JhcGhQZXJjZW50YWdlcyhyZWNvcmQpOwotICAgICAgICBpZiAocGVyY2Vu dGFnZXMuc3RhcnQgPCAxMDAgJiYgcGVyY2VudGFnZXMuZW5kV2l0aENoaWxkcmVuID49IDAgJiYg IXJlY29yZC5jYXRlZ29yeS5oaWRkZW4pIHsKLSAgICAgICAgICAgICsrdGhpcy5fcm9vdFJlY29y ZC5fdmlzaWJsZVJlY29yZHNDb3VudDsKLSAgICAgICAgICAgICsrcmVjb3JkLnBhcmVudC5faW52 aXNpYmxlQ2hpbGRyZW5Db3VudDsKLSAgICAgICAgICAgIGlmICghcGFyZW50SXNDb2xsYXBzZWQp Ci0gICAgICAgICAgICAgICAgcmVjb3Jkc1dpbmRvdy5wdXNoKHJlY29yZCk7Ci0gICAgICAgIH0K LQotICAgICAgICB2YXIgaW5kZXggPSByZWNvcmRzV2luZG93Lmxlbmd0aDsKLSAgICAgICAgcmVj b3JkLl9pbnZpc2libGVDaGlsZHJlbkNvdW50ID0gMDsKLSAgICAgICAgZm9yICh2YXIgaSA9IDA7 IGkgPCByZWNvcmQuY2hpbGRyZW4ubGVuZ3RoOyArK2kpCi0gICAgICAgICAgICB0aGlzLl9hZGRU b1JlY29yZHNXaW5kb3cocmVjb3JkLmNoaWxkcmVuW2ldLCByZWNvcmRzV2luZG93LCBwYXJlbnRJ c0NvbGxhcHNlZCB8fCByZWNvcmQuY29sbGFwc2VkKTsKLSAgICAgICAgcmVjb3JkLl92aXNpYmxl Q2hpbGRyZW5Db3VudCA9IHJlY29yZHNXaW5kb3cubGVuZ3RoIC0gaW5kZXg7Ci0gICAgfSwKLQog ICAgIF9maWx0ZXJSZWNvcmRzOiBmdW5jdGlvbigpCiAgICAgewogICAgICAgICB2YXIgcmVjb3Jk c0luV2luZG93ID0gW107CiAgICAgICAgIHRoaXMuX3Jvb3RSZWNvcmQuX3Zpc2libGVSZWNvcmRz Q291bnQgPSAwOwotICAgICAgICBmb3IgKHZhciBpID0gMDsgaSA8IHRoaXMuX3Jvb3RSZWNvcmQu Y2hpbGRyZW4ubGVuZ3RoOyArK2kpCi0gICAgICAgICAgICB0aGlzLl9hZGRUb1JlY29yZHNXaW5k b3codGhpcy5fcm9vdFJlY29yZC5jaGlsZHJlbltpXSwgcmVjb3Jkc0luV2luZG93KTsKKworICAg ICAgICB2YXIgc3RhY2sgPSBbe2NoaWxkcmVuOiB0aGlzLl9yb290UmVjb3JkLmNoaWxkcmVuLCBp bmRleDogMCwgcGFyZW50SXNDb2xsYXBzZWQ6IGZhbHNlfV07CisgICAgICAgIHdoaWxlIChzdGFj ay5sZW5ndGgpIHsKKyAgICAgICAgICAgIHZhciBlbnRyeSA9IHN0YWNrW3N0YWNrLmxlbmd0aCAt IDFdOworICAgICAgICAgICAgdmFyIHJlY29yZHMgPSBlbnRyeS5jaGlsZHJlbjsKKyAgICAgICAg ICAgIGlmIChyZWNvcmRzICYmIGVudHJ5LmluZGV4IDwgcmVjb3Jkcy5sZW5ndGgpIHsKKyAgICAg ICAgICAgICAgICAgdmFyIHJlY29yZCA9IHJlY29yZHNbZW50cnkuaW5kZXhdOworICAgICAgICAg ICAgICAgICArK2VudHJ5LmluZGV4OworCisgICAgICAgICAgICAgICAgIGlmICghdGhpcy5fc2hv d1Nob3J0RXZlbnRzICYmICFyZWNvcmQuaXNMb25nKCkpCisgICAgICAgICAgICAgICAgICAgICBj b250aW51ZTsKKyAgICAgICAgICAgICAgICAgdmFyIHBlcmNlbnRhZ2VzID0gdGhpcy5fY2FsY3Vs YXRvci5jb21wdXRlQmFyR3JhcGhQZXJjZW50YWdlcyhyZWNvcmQpOworICAgICAgICAgICAgICAg ICBpZiAocGVyY2VudGFnZXMuc3RhcnQgPCAxMDAgJiYgcGVyY2VudGFnZXMuZW5kV2l0aENoaWxk cmVuID49IDAgJiYgIXJlY29yZC5jYXRlZ29yeS5oaWRkZW4pIHsKKyAgICAgICAgICAgICAgICAg ICAgICsrdGhpcy5fcm9vdFJlY29yZC5fdmlzaWJsZVJlY29yZHNDb3VudDsKKyAgICAgICAgICAg ICAgICAgICAgICsrcmVjb3JkLnBhcmVudC5faW52aXNpYmxlQ2hpbGRyZW5Db3VudDsKKyAgICAg ICAgICAgICAgICAgICAgIGlmICghZW50cnkucGFyZW50SXNDb2xsYXBzZWQpCisgICAgICAgICAg ICAgICAgICAgICAgICAgcmVjb3Jkc0luV2luZG93LnB1c2gocmVjb3JkKTsKKyAgICAgICAgICAg ICAgICAgfQorCisgICAgICAgICAgICAgICAgIHJlY29yZC5faW52aXNpYmxlQ2hpbGRyZW5Db3Vu dCA9IDA7CisKKyAgICAgICAgICAgICAgICAgc3RhY2sucHVzaCh7Y2hpbGRyZW46IHJlY29yZC5j aGlsZHJlbiwKKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgaW5kZXg6IDAsCisgICAgICAg ICAgICAgICAgICAgICAgICAgICAgIHBhcmVudElzQ29sbGFwc2VkOiAoZW50cnkucGFyZW50SXND b2xsYXBzZWQgfHwgcmVjb3JkLmNvbGxhcHNlZCksCisgICAgICAgICAgICAgICAgICAgICAgICAg ICAgIHBhcmVudFJlY29yZDogcmVjb3JkLAorICAgICAgICAgICAgICAgICAgICAgICAgICAgICB3 aW5kb3dMZW5ndGhCZWZvcmVDaGlsZHJlblRyYXZlcnNhbDogcmVjb3Jkc0luV2luZG93Lmxlbmd0 aH0pOworICAgICAgICAgICAgfSBlbHNlIHsKKyAgICAgICAgICAgICAgICBzdGFjay5wb3AoKTsK KyAgICAgICAgICAgICAgICBpZiAoZW50cnkucGFyZW50UmVjb3JkKQorICAgICAgICAgICAgICAg ICAgICBlbnRyeS5wYXJlbnRSZWNvcmQuX3Zpc2libGVDaGlsZHJlbkNvdW50ID0gcmVjb3Jkc0lu V2luZG93Lmxlbmd0aCAtIGVudHJ5LndpbmRvd0xlbmd0aEJlZm9yZUNoaWxkcmVuVHJhdmVyc2Fs OworICAgICAgICAgICAgfQorICAgICAgICB9CisKICAgICAgICAgcmV0dXJuIHJlY29yZHNJbldp bmRvdzsKICAgICB9LAogCg==