6639 2006-01-17 20:52:55 -0800 for/in returns duplicate keys when key exists on object and its prototype 2007-09-04 14:42:32 -0700 1 1 1 Unclassified WebKit JavaScriptCore 420+ Mac OS X 10.4 RESOLVED DUPLICATE 9950 HasReduction P2 Normal --- 10489 1 jesse webkit-unassigned ap ian maksim mrowe tolmasky oldest_to_newest 28770 0 jesse 2006-01-17 20:52:55 -0800 The following code evaluates to "toString toString", however, for...in should never return duplicate keys. var A = function() {}; A.prototype.toString = function() { "A" }; var B = function() {}; B.prototype = new A(); B.prototype.toString = function() { "B" }; var s = ""; for (var f in B.prototype) s += f + " "; s 28771 1 jesse 2006-01-17 20:53:15 -0800 I am evaluating Safari for a port of TIBCO(R) General Interface, a mature AJAX platform that is currently IE only. Fixing this bug would make the port more feasible. 28785 2 5754 mrowe 2006-01-17 21:56:42 -0800 Created attachment 5754 Test case packged suitable for a layout test 28787 3 mrowe 2006-01-17 21:58:26 -0800 Thanks for the bug report. I can confirm this fails in both WebKit 417.9 and ToT WebKit. 33482 4 mjs 2006-02-20 06:42:51 -0800 I had a possible fix for this at one point but it caused an apparent performance regression. Maybe time to retest. 35512 5 6937 mrowe 2006-03-08 02:51:22 -0800 Created attachment 6937 Further simplified test case 35514 6 mrowe 2006-03-08 02:52:03 -0800 *** Bug 7658 has been marked as a duplicate of this bug. *** 50129 7 ap 2006-07-16 21:44:23 -0700 *** This bug has been marked as a duplicate of 9950 *** 45322 8 jesse 2006-11-18 17:22:33 -0800 Thanks to the Safari team for being so responsive regarding requests for better advanced AJAX support. Safari is quickly becoming a 1st class AJAX platform. 917 9 jesse 2007-09-04 14:42:32 -0700 This is fixed in Safari 3 for iPhone 1.0.2 5754 2006-01-17 21:56:42 -0800 2006-03-08 02:51:22 -0800 Test case packged suitable for a layout test webkit-bug-6639-test-v1.html text/html 1111 mrowe PGh0bWw+CiAgICA8aGVhZD4KICAgIDwvaGVhZD4KICAgIDxib2R5PgogICAgICAgIDxkaXYgaWQ9 Im91dHB1dCI+CiAgICAgICAgPC9kaXY+CiAgICAgICAgPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFz Y3JpcHQiPgogICAgICAgICAgICBmdW5jdGlvbiBzaG91bGRCZShkZXNjcmlwdGlvbiwgZXhwZWN0 ZWQsIGFjdHVhbCkKICAgICAgICAgICAgewogICAgICAgICAgICAgICAgdmFyIG1zZzsKICAgICAg ICAgICAgICAgIGlmIChleHBlY3RlZCA9PSBhY3R1YWwpCiAgICAgICAgICAgICAgICB7CiAgICAg ICAgICAgICAgICAgICAgbXNnID0gIlBBU1M6ICIgKyBkZXNjcmlwdGlvbjsKICAgICAgICAgICAg ICAgIH0KICAgICAgICAgICAgICAgIGVsc2UKICAgICAgICAgICAgICAgIHsKICAgICAgICAgICAg ICAgICAgICBtc2cgPSAiRkFJTDogIiArIGRlc2NyaXB0aW9uICsgIiAoZXhwZWN0ZWQgIiArIGV4 cGVjdGVkICsgIiwgZ290ICIgKyBhY3R1YWwgKyAiKSI7CiAgICAgICAgICAgICAgICB9CiAgICAg ICAgICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgib3V0cHV0IikuYXBwZW5kQ2hpbGQo ZG9jdW1lbnQuY3JlYXRlVGV4dE5vZGUobXNnKSk7CiAgICAgICAgICAgIH0KICAgICAgICAKICAg ICAgICAgICAgdmFyIEEgPSBmdW5jdGlvbigpIHt9OwogICAgICAgICAgICBBLnByb3RvdHlwZS50 b1N0cmluZyA9IGZ1bmN0aW9uKCkgeyByZXR1cm4gIkEiOyB9OwoKICAgICAgICAgICAgdmFyIEIg PSBmdW5jdGlvbigpIHt9OwogICAgICAgICAgICBCLnByb3RvdHlwZSA9IG5ldyBBKCk7CiAgICAg ICAgICAgIEIucHJvdG90eXBlLnRvU3RyaW5nID0gZnVuY3Rpb24oKSB7IHJldHVybiAiQiI7IH07 CgogICAgICAgICAgICB2YXIgZW50cmllcyA9IFtdOwogICAgICAgICAgICBmb3IgKHZhciBmIGlu IEIucHJvdG90eXBlKQogICAgICAgICAgICAgICAgZW50cmllcy5wdXNoKGYpOwogICAgICAgICAg ICBzaG91bGRCZSgiUHJvdG90eXBlIHNob3VsZCBub3QgaGF2ZSBkdXBsaWNhdGUgZW50cmllcyBm b3IgdG9TdHJpbmciLCAidG9TdHJpbmciLCBlbnRyaWVzLnRvU3RyaW5nKCkpOwogICAgICAgIDwv c2NyaXB0PgogICAgPC9ib2R5Pgo8L2h0bWw+Cg== 6937 2006-03-08 02:51:22 -0800 2006-03-08 02:51:22 -0800 Further simplified test case webkit-bug-6639-2.html text/html 931 mrowe PGh0bWw+CiAgICA8aGVhZD4KICAgIDwvaGVhZD4KICAgIDxib2R5PgogICAgICAgIDxkaXYgaWQ9 Im91dHB1dCI+CiAgICAgICAgPC9kaXY+CiAgICAgICAgPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFz Y3JpcHQiPgogICAgICAgICAgICBmdW5jdGlvbiBzaG91bGRCZShkZXNjcmlwdGlvbiwgZXhwZWN0 ZWQsIGFjdHVhbCkKICAgICAgICAgICAgewogICAgICAgICAgICAgICAgdmFyIG1zZzsKICAgICAg ICAgICAgICAgIGlmIChleHBlY3RlZCA9PSBhY3R1YWwpCiAgICAgICAgICAgICAgICAgICAgbXNn ID0gIlBBU1M6ICIgKyBkZXNjcmlwdGlvbjsKICAgICAgICAgICAgICAgIGVsc2UKICAgICAgICAg ICAgICAgICAgICBtc2cgPSAiRkFJTDogIiArIGRlc2NyaXB0aW9uICsgIiAoZXhwZWN0ZWQgIiAr IGV4cGVjdGVkICsgIiwgZ290ICIgKyBhY3R1YWwgKyAiKSI7CiAgICAgICAgICAgICAgICBkb2N1 bWVudC5nZXRFbGVtZW50QnlJZCgib3V0cHV0IikuYXBwZW5kQ2hpbGQoZG9jdW1lbnQuY3JlYXRl VGV4dE5vZGUobXNnKSk7CiAgICAgICAgICAgIH0KCiAgICAgICAgICAgIHZhciBhQ2xhc3MgPSBm dW5jdGlvbigpIHt9OwogICAgICAgICAgICBhQ2xhc3MucHJvdG90eXBlLmZvbyA9IGZ1bmN0aW9u KCkge307CiAgICAgICAgICAgIAogICAgICAgICAgICB2YXIgYSA9IG5ldyBhQ2xhc3MoKTsKICAg ICAgICAgICAgYS5mb28gPSAxOwoKICAgICAgICAgICAgdmFyIGVudHJpZXMgPSBbXTsKICAgICAg ICAgICAgZm9yICh2YXIgZiBpbiBhKQogICAgICAgICAgICAgICAgZW50cmllcy5wdXNoKGYpOwog ICAgICAgICAgICBzaG91bGRCZSgiT2JqZWN0IHNob3VsZCBub3QgaGF2ZSBkdXBsaWNhdGUgcHJv cGVydGllcyIsICJmb28iLCBlbnRyaWVzLnRvU3RyaW5nKCkpOwogICAgICAgIDwvc2NyaXB0Pgog ICAgPC9ib2R5Pgo8L2h0bWw+Cg==