53566 2011-02-01 18:15:00 -0800 [v8] Increase V8 native->js recursion limit to match document.write() recursion limit 2011-04-01 13:24:22 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Other OS X 10.5 RESOLVED FIXED P2 Normal --- 1 jamesr jamesr abarth ager annacc barraclough commit-queue dglazkov eric webkit.review.bot oldest_to_newest 343743 0 jamesr 2011-02-01 18:15:00 -0800 [v8] Increase V8 native->js recursion limit to match document.write() recursion limit 343747 1 80869 jamesr 2011-02-01 18:16:23 -0800 Created attachment 80869 Patch 343749 2 jamesr 2011-02-01 18:18:07 -0800 This passes the test. I would prefer to match JSC exactly here but I can't find any equivalent limit in the JSC bindings, at least at a quick glance. Adding some people who might be familiar with the JSC bindings to see if they know of such a limit. 343793 3 80869 abarth 2011-02-01 19:55:09 -0800 Comment on attachment 80869 Patch In crbug, you said JSC's limit was 256. I'll leave it up to your judgement as to which number to use. (I'd probably go with 22 absent other information.) 343796 4 barraclough 2011-02-01 20:04:23 -0800 JSC should soon be removing its arbitrary limit in favour of checking available machine stack, so matching the current limit is probably less important. (FWIW 22 sounds like a perfectly reasonable limit to me too!) 343819 5 80869 jamesr 2011-02-01 21:13:13 -0800 Comment on attachment 80869 Patch Let's go with 22 and see what happens! 343874 6 80869 commit-queue 2011-02-01 23:03:03 -0800 Comment on attachment 80869 Patch Clearing flags on attachment: 80869 Committed r77364: <http://trac.webkit.org/changeset/77364> 343875 7 commit-queue 2011-02-01 23:03:09 -0800 All reviewed patches have been landed. Closing bug. 343889 8 webkit.review.bot 2011-02-01 23:35:40 -0800 http://trac.webkit.org/changeset/77364 might have broken SnowLeopard Intel Release (Tests) The following tests are not passing: http/tests/xmlhttprequest/cross-origin-no-authorization.html 378030 9 annacc 2011-04-01 13:24:22 -0700 *** Bug 53557 has been marked as a duplicate of this bug. *** 80869 2011-02-01 18:16:23 -0800 2011-02-01 23:03:03 -0800 Patch bug-53566-20110201181622.patch text/plain 2743 jamesr ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCAwMTlhYjQwYmJjMDdmOGZmMjljOTY4MzQ1MjQ4Mjc5NjBlZjJkNGU5Li5hYzBmOWJl MWJkMDI2NDBjMzFhYmYzMGU2OWI1M2Y0ZTA0MjVkYjQ3IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0 cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAK KzIwMTEtMDItMDEgIEphbWVzIFJvYmluc29uICA8amFtZXNyQGNocm9taXVtLm9yZz4KKworICAg ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbdjhdIEluY3JlYXNl IFY4IG5hdGl2ZS0+anMgcmVjdXJzaW9uIGxpbWl0IHRvIG1hdGNoIGRvY3VtZW50LndyaXRlKCkg cmVjdXJzaW9uIGxpbWl0CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn LmNnaT9pZD01MzU2NgorCisgICAgICAgIFVuc2tpcCB0ZXN0IHRoYXQgd2Ugbm93IHBhc3MuCisK KyAgICAgICAgKiBwbGF0Zm9ybS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQ6CisKIDIw MTEtMDItMDEgIEtlbnQgVGFtdXJhICA8dGtlbnRAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJl dmlld2VkIGJ5IFRvbnkgQ2hhbmcuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9j aHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJv bWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQKaW5kZXggNGI5N2Y3ZGQ0MWJiYTI4OTZhODliM2I5 NTM4MTRiZTQxNGM1ZDU5Mi4uN2Y4ZjcxYTgzYmU4ZGNmOTBmM2ViYTIwNmZkMjRiZTY1YWY3OGIw NCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vY2hyb21pdW0vdGVzdF9leHBlY3Rh dGlvbnMudHh0CisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0 YXRpb25zLnR4dApAQCAtMzA4NCw0ICszMDg0LDMgQEAgQlVHV0s1MzUwNSBXSU4gREVCVUcgOiBp bnNwZWN0b3IvY29uc29sZS1kaXIuaHRtbCA9IFBBU1MgQ1JBU0gKIEJVR1dLNTM1MDUgV0lOIERF QlVHIDogaW5zcGVjdG9yL2NwdS1wcm9maWxlci1wcm9maWxpbmcuaHRtbCA9IFBBU1MgQ1JBU0gK IAogQlVHV0s1MzUyNyBERUJVRyA6IGZhc3QvZnJhbWVzL2lmcmFtZS1yZXBhcmVudGluZy1uZXct cGFnZS5odG1sID0gQ1JBU0ggUEFTUwotQlVHQ1I3MTYzMiBTS0lQIDogZmFzdC9kb20vRG9jdW1l bnQvZG9jdW1lbnQtd3JpdGUtcmVjdXJzaW9uLmh0bWwgPSBUSU1FT1VUCmRpZmYgLS1naXQgYS9T b3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg NTAxNWFhN2UwYTUyYzk4MDcyZDQ1Zjk3YmRjZmRkZDQwNzYwNWFiNy4uMjM3MGYzMTMzZmZkOGFj Y2VhNzI0ZjY1MzcxYWMwOWM1YjI5YzQ1ZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hh bmdlTG9nCisrKyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisy MDExLTAyLTAxICBKYW1lcyBSb2JpbnNvbiAgPGphbWVzckBjaHJvbWl1bS5vcmc+CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW3Y4XSBJbmNyZWFzZSBW OCBuYXRpdmUtPmpzIHJlY3Vyc2lvbiBsaW1pdCB0byBtYXRjaCBkb2N1bWVudC53cml0ZSgpIHJl Y3Vyc2lvbiBsaW1pdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j Z2k/aWQ9NTM1NjYKKworICAgICAgICBBIHJlY3Vyc2lvbiBsaW1pdCBvZiAyMiBpcyBuZWNlc3Nh cnkgdG8gcGFzcyBmYXN0L2RvbS9Eb2N1bWVudC9kb2N1bWVudC13cml0ZS1yZWN1cnNpb24uaHRt bC4KKyAgICAgICAgT3RoZXIgdGhhbiBiZWluZyBsYXJnZSBlbm91Z2ggZm9yIHRoaXMgb25lIHRl c3QgY2FzZSwgdGhpcyBsaW1pdCBpcyBhcmJpdHJhcnkuCisKKyAgICAgICAgKiBiaW5kaW5ncy92 OC9WOFByb3h5Lmg6CisKIDIwMTEtMDEtMjUgIE1hcnRpbiBSb2JpbnNvbiAgPG1yb2JpbnNvbkBp Z2FsaWEuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEd1c3Rhdm8gTm9yb25oYSBTaWx2YS4K ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4UHJveHkuaCBiL1NvdXJj ZS9XZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4UHJveHkuaAppbmRleCAzZTdmMzdhODA0NzM1YzUxNGEy ZDUyNDY4OGE2MjY5NTJhMmU3Zjc2Li43Nzc4MTViNmZmZDA5MmYyZmRkMDJhMzBlZTQxMGJmMzg0 NTU3YTBhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9iaW5kaW5ncy92OC9WOFByb3h5LmgK KysrIGIvU291cmNlL1dlYkNvcmUvYmluZGluZ3MvdjgvVjhQcm94eS5oCkBAIC0xMTYsNyArMTE2 LDcgQEAgbmFtZXNwYWNlIFdlYkNvcmUgewogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgY29uc3QgQmF0Y2hlZENhbGxiYWNrKiwgCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICBzaXplX3QgY2FsbGJhY2tDb3VudCk7CiAKLSAgICBjb25zdCBpbnQga01heFJlY3Vyc2lv bkRlcHRoID0gMjA7CisgICAgY29uc3QgaW50IGtNYXhSZWN1cnNpb25EZXB0aCA9IDIyOwogCiAg ICAgLy8gVGhlIGxpc3Qgb2YgZXh0ZW5zaW9ucyB0aGF0IGFyZSByZWdpc3RlcmVkIGZvciB1c2Ug d2l0aCBWOC4KICAgICB0eXBlZGVmIFdURjo6VmVjdG9yPHY4OjpFeHRlbnNpb24qPiBWOEV4dGVu c2lvbnM7Cg==