146555 2015-07-02 12:26:09 -0700 Relax builtin JS restriction about try-catch 2015-07-03 10:39:24 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 146229 1 ysuzuki ysuzuki commit-queue darin fpizlo ggaren mark.lam msaboff oliver sam oldest_to_newest 1106436 0 ysuzuki 2015-07-02 12:26:09 -0700 try-catch cannot be used in builtin JS code. But relaxing this restriction is useful for writing Promise function in JS. 1106613 1 256070 ysuzuki 2015-07-02 21:33:27 -0700 Created attachment 256070 Patch 1106719 2 ysuzuki 2015-07-03 09:30:47 -0700 Could anyone take a look? 1106721 3 256070 ysuzuki 2015-07-03 09:48:14 -0700 Comment on attachment 256070 Patch Thanks for your review! 1106730 4 256070 commit-queue 2015-07-03 10:39:20 -0700 Comment on attachment 256070 Patch Clearing flags on attachment: 256070 Committed r186260: <http://trac.webkit.org/changeset/186260> 1106731 5 commit-queue 2015-07-03 10:39:24 -0700 All reviewed patches have been landed. Closing bug. 256070 2015-07-02 21:33:27 -0700 2015-07-03 10:39:20 -0700 Patch bug-146555-20150703133232.patch text/plain 2314 ysuzuki U3VidmVyc2lvbiBSZXZpc2lvbjogMTg2MjI2CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCAy MTkwNjRlNjkyMzkyMGEzOTRmMWJiNjk5MzY5ZjlhYWU4ODY3OWY0Li5mNDljMjExODQzNmExOWQz NDU3ZmQ2ZjZkNzA1MTFlMjQ4MThkYjRlIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs MyArMSwyNyBAQAorMjAxNS0wNy0wMiAgWXVzdWtlIFN1enVraSAgPHV0YXRhbmUudGVhQGdtYWls LmNvbT4KKworICAgICAgICBSZWxheCBidWlsdGluIEpTIHJlc3RyaWN0aW9uIGFib3V0IHRyeS1j YXRjaAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTQ2 NTU1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2hl biByZXRyaWV2aW5nIHRoZSBjYXB0dXJlZCB2YXJpYWJsZXMgZnJvbSB0aGUgZnVsbCBhY3RpdmF0 ZWQgc2NvcGUsCisgICAgICAgIGl0IHN3YXBwZWQgdGhlIGdpdmVuIHZlY3RvciB3aXRoIHRoZSBz dG9yZWQgZGVjbGFyZWQgdmFyaWFibGVzIHZlY3Rvci4KKyAgICAgICAgVGhpcyBpcyBiZWNhdXNl IHJldHJpZXZpbmcgdGhlIGNhcHR1cmVkIHZhcmlhYmxlcyBhcmUgZXhlY3V0ZWQgaW4gdGhlCisg ICAgICAgIGxhc3Qgc2VxdWVuY2Ugb2YgdGhlIHBhcnNlciwgc28gZGVjbGFyZWQgdmFyaWFibGVz IGFyZSBubyBsb25nZXIgdXNlZC4KKyAgICAgICAgSG93ZXZlciwgaW4gYnVpbHRpbnMgZnVuY3Rp b25zIGNhc2UsIGFmdGVyIHJldHJpZXZpbmcgdGhlIGNhcHR1cmVkCisgICAgICAgIHZhcmlhYmxl cywgd2UgY2hlY2sgdGhlIHZhcmlhYmxlcyBieSB1c2luZyBkZWNsYXJlZCB2YXJpYWJsZXMgdmVj dG9yLgorICAgICAgICBTbyBhdCB0aGF0IHRpbWUsIHRoZSBkZWNsYXJlZCB2YXJpYWJsZXMgdmVj dG9yIGJlY29tZXMgZW1wdHkgYW5kIGl0CisgICAgICAgIHJhaXNlcyBhc3NlcnRpb24gZmFpbHVy ZXMgd2hlbiB0aGUgYnVpbHRpbnMgZnVuY3Rpb24gY29udGFpbnMgdGhlIGZ1bGwKKyAgICAgICAg YWN0aXZhdGVkIHNjb3BlLiB0cnktY2F0Y2gncyBjYXRjaCBzY29wZSByZXF1aXJlcyB0aGUgdXBw ZXIgc2NvcGUgZnVsbAorICAgICAgICBhY3RpdmF0ZWQsIHNvIEpTIGNvZGUgaW4gdGhlIGJ1aWx0 aW5zIGNhbm5vdCB1c2UgdGhlIHRyeS1jYXRjaC4KKworICAgICAgICBUaGlzIHBhdGNoIHJlbGF4 ZXMgdGhpcyByZXN0cmljdGlvbi4gV2hlbiByZXRyaWV2aW5nIHRoZSBjYXB0dXJlZAorICAgICAg ICB2YXJpYWJsZXMgZnJvbSB0aGUgc2NvcGUsIGp1c3QgY29weSB0byB0aGUgZ2l2ZW4gdmVjdG9y LgorCisgICAgICAgICogcGFyc2VyL1BhcnNlci5oOgorICAgICAgICAoSlNDOjpTY29wZTo6Z2V0 Q2FwdHVyZWRWYXJpYWJsZXMpOgorCiAyMDE1LTA3LTAyICBEYW4gQmVybnN0ZWluICA8bWl0ekBh cHBsZS5jb20+CiAKICAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzIxNDI5NjEzPiBbaU9TXSBTdG9w IG1ha2luZyBzeW1saW5rcyBmcm9tIFByaXZhdGVGcmFtZXdvcmtzIHRvIEZyYW1ld29ya3MKZGlm ZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9wYXJzZXIvUGFyc2VyLmggYi9Tb3VyY2Uv SmF2YVNjcmlwdENvcmUvcGFyc2VyL1BhcnNlci5oCmluZGV4IDJjNjA1ZTZjNjIxYzU5NjhkMTFj YzM4ZGMxZjQ1MWMzNTk2ODE0Y2YuLjA1OGFhMjUyNjBjYzE1MTU2YjFhZjExZWM0MmVjM2VhZDRi MDM0ZDEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9wYXJzZXIvUGFyc2VyLmgK KysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL3BhcnNlci9QYXJzZXIuaApAQCAtMzExLDcgKzMx MSw3IEBAIHN0cnVjdCBTY29wZSB7CiAgICAgewogICAgICAgICBpZiAobV9uZWVkc0Z1bGxBY3Rp dmF0aW9uIHx8IG1fdXNlc0V2YWwpIHsKICAgICAgICAgICAgIG1vZGlmaWVkUGFyYW1ldGVyID0g dHJ1ZTsKLSAgICAgICAgICAgIGNhcHR1cmVkVmFyaWFibGVzLnN3YXAobV9kZWNsYXJlZFZhcmlh Ymxlcyk7CisgICAgICAgICAgICBjYXB0dXJlZFZhcmlhYmxlcyA9IG1fZGVjbGFyZWRWYXJpYWJs ZXM7CiAgICAgICAgICAgICByZXR1cm47CiAgICAgICAgIH0KICAgICAgICAgZm9yIChJZGVudGlm aWVyU2V0OjppdGVyYXRvciBwdHIgPSBtX2Nsb3NlZFZhcmlhYmxlcy5iZWdpbigpOyBwdHIgIT0g bV9jbG9zZWRWYXJpYWJsZXMuZW5kKCk7ICsrcHRyKSB7Cg==