6696 2006-01-21 09:21:08 -0800 REGRESSION: JavaScript window.opener set incorrectly 2006-01-31 21:20:55 -0800 1 1 1 Unclassified WebKit DOM 420+ Mac OS X 10.4 VERIFIED FIXED http://www.emprint.fr/webkit/test.html P1 Normal --- 1 julien darin ddkilzer oldest_to_newest 29256 0 julien 2006-01-21 09:21:08 -0800 This bug happen only in WebKit and not in Safari (417.8). When you open a new window using Javascript [window.open(...)], the new window doesn't like to interpret window.opener as the opener window but as itself. We can't send values from one window to another. You can check this at http://www.emprint.fr/webkit/test.html 29258 1 ap 2006-01-21 09:54:21 -0800 Confirmed; setting priority to P1 (regression). 29260 2 joost 2006-01-21 10:05:57 -0800 Adding regression keyword and to summary. 29656 3 ddkilzer 2006-01-23 21:27:16 -0800 I'm pretty sure this bug is what's causing Yahoo! Mail's "insert address" feature to stop working. 1. Log in to Yahoo! Mail using Safari+WebKit. 2. Click on "Compose" button. 3. Click on "To:" link. 4. Select an address to add (check a checkbox in the "To" column). 5. Click on "Insert Checked Contacts" button. Expected: The address should be added to the "To:" text box. Actual: Nothing happens. 29657 4 5897 ddkilzer 2006-01-23 21:32:10 -0800 Created attachment 5897 Target window contents (click on other test file) 29658 5 5898 ddkilzer 2006-01-23 21:33:15 -0800 Created attachment 5898 Test file (click here first) 29666 6 ddkilzer 2006-01-23 21:59:02 -0800 Test fails: WebKit-SVN-r11976.dmg Tue Jan 10 4:35:03 GMT 2006 Test works: WebKit-CVS-2006-01-07 09:37:19 GMT.dmg Regression was checked in between revisions r11922 and r11976, inclusive. 29693 7 ddkilzer 2006-01-24 03:52:12 -0800 After further testing, it appears that in ToT WebKit that this is true for the pop-up window, which it shouldn't be: window.opener == window 29729 8 darin 2006-01-24 12:05:50 -0800 This was a regression from the Frame change, bug 6450. I have a fix. 29735 9 5924 darin 2006-01-24 12:58:31 -0800 Created attachment 5924 patch that fixes the name collision that created this problem 29821 10 julien 2006-01-25 02:13:39 -0800 Thank you :) It's ok in the new release. 29827 11 ddkilzer 2006-01-25 04:54:15 -0800 Marking verified per Comment #10. 30892 12 eric 2006-01-31 21:20:55 -0800 Removing Regression keyword from bugs already fixed. 5897 2006-01-23 21:32:10 -0800 2006-01-23 21:32:10 -0800 Target window contents (click on other test file) test1.html text/html 382 ddkilzer PGh0bWw+CiAgICA8aGVhZD4KICAgICAgICA8c2NyaXB0PgogICAgICAgICAgICBmdW5jdGlvbiBz ZW5kQmFjaygpCiAgICAgICAgICAgIHsKICAgICAgICAgICAgICAgIHdpbmRvdy5vcGVuZXIuZG9j dW1lbnQubXlGb3JtLm15SW5wdXQudmFsdWUgPSAnSGVsbG8nOwogICAgICAgICAgICB9CiAgICAg ICAgPC9zY3JpcHQ+CiAgICA8L2hlYWQ+CiAgICA8Ym9keT4KICAgICAgICA8Zm9ybT4KICAgICAg ICA8aW5wdXQgdHlwZT0nYnV0dG9uJyBuYW1lPSdhQnV0dG9uJyB2YWx1ZT0iQ2xpY2sgaGVyZSB0 byBzZW5kICdIZWxsbycgdG8gdGhlIG9wZW5lciB3aW5kb3ciIG9uQ2xpY2s9J3NlbmRCYWNrKCk7 JyAvPgogICAgICAgIDwvZm9ybT4KICAgIDwvYm9keT4KPC9odG1sPg== 5898 2006-01-23 21:33:15 -0800 2006-01-23 21:33:15 -0800 Test file (click here first) test.html text/html 484 ddkilzer PGh0bWw+CiAgICA8aGVhZD4KICAgICAgICA8c2NyaXB0IGxhbmd1YWdlPSJKYXZhU2NyaXB0Ij4K ICAgICAgICA8IS0tCiAgICAgICAgICAgIGZ1bmN0aW9uIG9wZW5XaW5kb3coKSB7CiAgICAgICAg ICAgICAgICBteVdpbmRvdyA9IHdpbmRvdy5vcGVuKCJhdHRhY2htZW50LmNnaT9pZD01ODk3Iiwg InRlc3RXaW5kb3ciLCAid2lkdGg9NDAwLGhlaWdodD0yMDAiKTsKICAgICAgICAgICAgfQoKICAg ICAgICAvLyAtLT4KICAgICAgICA8L3NjcmlwdD4KICAgIDwvaGVhZD4KICAgIDxib2R5PgoKICAg ICAgICA8Zm9ybSBuYW1lPSJteUZvcm0iPgogICAgICAgICAgICA8aW5wdXQgdHlwZT0idGV4dCIg bmFtZT0ibXlJbnB1dCIgc2l6ZT0xMiBtYXhsZW5ndGg9MTIgLz4mbmJzcDs8aW5wdXQgdHlwZT0i YnV0dG9uIiB2YWx1ZT0iT3BlbiBJdCIgb25DbGljaz0ib3BlbldpbmRvdygpOyI+CiAgICAgICAg PC9mb3JtPgogICAgPC9ib2R5Pgo8L2h0bWw+Cg== 5924 2006-01-24 12:58:31 -0800 2006-01-24 15:00:11 -0800 patch that fixes the name collision that created this problem WindowOpenerPatch.txt text/plain 3462 darin SW5kZXg6IGtodG1sL2VjbWEva2pzX3dpbmRvdy5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0ga2h0bWwvZWNt YS9ranNfd2luZG93LmNwcAkocmV2aXNpb24gMTIzMzYpCisrKyBraHRtbC9lY21hL2tqc193aW5k b3cuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNjMyLDQwICsxNjMyLDM2IEBACiAgICAgICB1YXJn cy5zZXJ2aWNlVHlwZSA9ICJ0ZXh0L2h0bWwiOwogICAgICAgCiAgICAgICAvLyByZXF1ZXN0IHdp bmRvdyAobmV3IG9yIGV4aXN0aW5nIGlmIGZyYW1lbmFtZSBpcyBzZXQpCi0gICAgICBPYmplY3RD b250ZW50cyAqbmV3UGFydCA9IDBMOworICAgICAgT2JqZWN0Q29udGVudHMqIG5ld1BhcnQgPSAw OwogICAgICAgdWFyZ3MubWV0YURhdGEoKS5zZXQoInJlZmVycmVyIiwgYWN0aXZlUGFydC0+cmVm ZXJyZXIoKSk7CiAgICAgICBmcmFtZS0+YnJvd3NlckV4dGVuc2lvbigpLT5jcmVhdGVOZXdXaW5k b3coIiIsIHVhcmdzLCB3aW5kb3dBcmdzLCBuZXdQYXJ0KTsKICAgICAgIGlmICghbmV3UGFydCB8 fCAhbmV3UGFydC0+aW5oZXJpdHMoIkZyYW1lIikpCiAgICAgICAgICAgcmV0dXJuIGpzVW5kZWZp bmVkKCk7Ci0gICAgICBGcmFtZSAqZnJhbWUgPSBzdGF0aWNfY2FzdDxGcmFtZSo+KG5ld1BhcnQp OwotICAgICAgLy9xRGVidWcoIm9wZW5lciBzZXQgdG8gJXAgKHRoaXMgV2luZG93J3MgZnJhbWUp IGluIG5ldyBXaW5kb3cgJXAgICh0aGlzIFdpbmRvdz0lcCkiLCBmcmFtZSx3aW4sd2luZG93KTsK LSAgICAgIGZyYW1lLT5zZXRPcGVuZXIoZnJhbWUpOwotICAgICAgZnJhbWUtPnNldE9wZW5lZEJ5 SlModHJ1ZSk7CisgICAgICBGcmFtZSAqbmV3RnJhbWUgPSBzdGF0aWNfY2FzdDxGcmFtZSo+KG5l d1BhcnQpOworICAgICAgbmV3RnJhbWUtPnNldE9wZW5lcihmcmFtZSk7CisgICAgICBuZXdGcmFt ZS0+c2V0T3BlbmVkQnlKUyh0cnVlKTsKICAgICAgIAotICAgICAgaWYgKCFmcmFtZS0+ZG9jdW1l bnQoKSkgewotICAgICAgICAgIERvY3VtZW50SW1wbCAqb2xkRG9jID0gZnJhbWUtPmRvY3VtZW50 KCk7CisgICAgICBpZiAoIW5ld0ZyYW1lLT5kb2N1bWVudCgpKSB7CisgICAgICAgICAgRG9jdW1l bnRJbXBsKiBvbGREb2MgPSBmcmFtZS0+ZG9jdW1lbnQoKTsKICAgICAgICAgICBpZiAob2xkRG9j ICYmIG9sZERvYy0+YmFzZVVSTCgpICE9IDApCi0gICAgICAgICAgICAgIGZyYW1lLT5iZWdpbihv bGREb2MtPmJhc2VVUkwoKSk7CisgICAgICAgICAgICAgIG5ld0ZyYW1lLT5iZWdpbihvbGREb2Mt PmJhc2VVUkwoKSk7CiAgICAgICAgICAgZWxzZQotICAgICAgICAgICAgICBmcmFtZS0+YmVnaW4o KTsKLSAgICAgICAgICAKLSAgICAgICAgICBmcmFtZS0+d3JpdGUoIjxIVE1MPjxCT0RZPiIpOwot ICAgICAgICAgIGZyYW1lLT5lbmQoKTsKLSAgICAgICAgICAKKyAgICAgICAgICAgICAgbmV3RnJh bWUtPmJlZ2luKCk7CisgICAgICAgICAgbmV3RnJhbWUtPndyaXRlKCI8SFRNTD48Qk9EWT4iKTsK KyAgICAgICAgICBuZXdGcmFtZS0+ZW5kKCk7ICAgICAgICAgIAogICAgICAgICAgIGlmIChvbGRE b2MpIHsKLSAgICAgICAgICAgICAga2REZWJ1Zyg2MDcwKSA8PCAiU2V0dGluZyBkb21haW4gdG8g IiA8PCBvbGREb2MtPmRvbWFpbigpLnFzdHJpbmcoKSA8PCBlbmRsOwotICAgICAgICAgICAgICBm cmFtZS0+ZG9jdW1lbnQoKS0+c2V0RG9tYWluKCBvbGREb2MtPmRvbWFpbigpLCB0cnVlICk7Ci0g ICAgICAgICAgICAgIGZyYW1lLT5kb2N1bWVudCgpLT5zZXRCYXNlVVJMKCBvbGREb2MtPmJhc2VV UkwoKSApOworICAgICAgICAgICAgICBuZXdGcmFtZS0+ZG9jdW1lbnQoKS0+c2V0RG9tYWluKG9s ZERvYy0+ZG9tYWluKCksIHRydWUpOworICAgICAgICAgICAgICBuZXdGcmFtZS0+ZG9jdW1lbnQo KS0+c2V0QmFzZVVSTChvbGREb2MtPmJhc2VVUkwoKSk7CiAgICAgICAgICAgfQogICAgICAgfQog ICAgICAgaWYgKCF1cmwuaXNFbXB0eSgpKSB7Ci0gICAgICAgICAgY29uc3QgV2luZG93KiB3aW5k b3cgPSBXaW5kb3c6OnJldHJpZXZlV2luZG93KGZyYW1lKTsKKyAgICAgICAgICBjb25zdCBXaW5k b3cqIHdpbmRvdyA9IFdpbmRvdzo6cmV0cmlldmVXaW5kb3cobmV3RnJhbWUpOwogICAgICAgICAg IGlmICghdXJsLnVybCgpLnN0YXJ0c1dpdGgoImphdmFzY3JpcHQ6IiwgZmFsc2UpIHx8ICh3aW5k b3cgJiYgd2luZG93LT5pc1NhZmVTY3JpcHQoZXhlYykpKSB7CiAgICAgICAgICAgICAgIGJvb2wg dXNlckdlc3R1cmUgPSBzdGF0aWNfY2FzdDxTY3JpcHRJbnRlcnByZXRlciAqPihleGVjLT5keW5h bWljSW50ZXJwcmV0ZXIoKSktPndhc1J1bkJ5VXNlckdlc3R1cmUoKTsKLSAgICAgICAgICAgICAg ZnJhbWUtPnNjaGVkdWxlTG9jYXRpb25DaGFuZ2UodXJsLnVybCgpLCBhY3RpdmVQYXJ0LT5yZWZl cnJlcigpLCBmYWxzZSwgdXNlckdlc3R1cmUpOwotICAgICAgICAgIH0gCisgICAgICAgICAgICAg IG5ld0ZyYW1lLT5zY2hlZHVsZUxvY2F0aW9uQ2hhbmdlKHVybC51cmwoKSwgYWN0aXZlUGFydC0+ cmVmZXJyZXIoKSwgZmFsc2UsIHVzZXJHZXN0dXJlKTsKKyAgICAgICAgICB9CiAgICAgICB9Ci0g ICAgICByZXR1cm4gV2luZG93OjpyZXRyaWV2ZShmcmFtZSk7IC8vIGdsb2JhbCBvYmplY3QKKyAg ICAgIHJldHVybiBXaW5kb3c6OnJldHJpZXZlKG5ld0ZyYW1lKTsgLy8gZ2xvYmFsIG9iamVjdAog ICB9CiAgIGNhc2UgV2luZG93OjpQcmludDoKICAgICBmcmFtZS0+cHJpbnQoKTsKSW5kZXg6IENo YW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDEyMzM2KQorKysgQ2hh bmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKK24gdDIwMDYtMDEtMjQgIERh cmluIEFkbGVyICA8ZGFyaW5AYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E WSAoT09QUyEpLgorCisgICAgICAgIC0gZml4IGh0dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9y Zy9zaG93X2J1Zy5jZ2k/aWQ9NjY5NgorICAgICAgICAgIFJFR1JFU1NJT046IEphdmFTY3JpcHQg d2luZG93Lm9wZW5lciBzZXQgaW5jb3JyZWN0bHkKKworICAgICAgICAqIGtodG1sL2VjbWEva2pz X3dpbmRvdy5jcHA6IChLSlM6OldpbmRvd0Z1bmM6OmNhbGxBc0Z1bmN0aW9uKToKKyAgICAgICAg Rml4IGxvY2FsIHZhcmlhYmxlICJmcmFtZSIgdGhhdCB3YXMgc2hhZG93aW5nIGFub3RoZXIgbG9j YWwgdmFyaWFibGUKKyAgICAgICAgbmFtZWQgImZyYW1lIiBieSBuYW1pbmcgdGhlIG9uZSBmb3Ig dGhlIG5ldyBmcmFtZSAibmV3RnJhbWUiLgorCiAyMDA2LTAxLTI0ICBNYWNpZWogU3RhY2hvd2lh ayAgPG1qc0BhcHBsZS5jb20+CiAKICAgICAgICAgLSBmaXggYnVpbGQK