26520 2009-06-18 15:17:25 -0700 Accessing a DOM node from a parent window in a child iframe moves the prototype to the child iframe and resets it in the parent window 2010-03-01 18:25:27 -0800 1 1 1 Unclassified WebKit WebCore JavaScript 528+ (Nightly build) All All RESOLVED FIXED http://paulsowden.com/etc/webkit/test-prototype.html P2 Normal --- 1 paul ap abarth ap ggaren gianni kangax sam oldest_to_newest 126788 0 paul 2009-06-18 15:17:25 -0700 In the test case we extend the prototype of HTMLInputElement to add a property "foo". In the child iframe I extend the prototype of HTMLInputElement to add a property "bar". In the child iframe we hook up an event listener to the parent window's body and access e.target. When clicking the submit button we should expect to see "foo", which is the property we added to the HTMLInputElement's prototype, but instead we see "undefined". Removing the line in the iframe which accesses e.target causes the alert to display "foo", as expected. It looks like when e.target is accessed in the child iframe it creates the HTMLElement DOM wrapper and this causes the prototypes from the parent window to be moved to the child iframe, then when we handle the onsubmit event in the parent window it lazyilly creates the HTMLElement DOM wrapper but with a blank prototype that now exists in the parent window. The extended prototypes are getting incorrectly moved between the two window objects. This looks like it could be related to bug 24795 but the testcase attached to that bug seems to work in the latest nightly so I suspect that it could actually be a different issue. 127846 1 ap 2009-06-24 00:44:36 -0700 Confirming as a difference with Firefox. 127871 2 abarth 2009-06-24 01:48:11 -0700 This is a dup of another bug that I can't put my fingers on immediately. This is also related to a recent thread on webkit-dev. 194426 3 ap 2010-02-26 19:51:08 -0800 The bug URL is no longer valid. Could you please attach the test case to the bug? 194438 4 49673 paul 2010-02-26 20:32:30 -0800 Created attachment 49673 Testcase for the main page (requires test-prototype-iframe.html as well) 194439 5 49674 paul 2010-02-26 20:33:16 -0800 Created attachment 49674 Testcase for the iframe (is loaded by the first testcase page) 194448 6 ap 2010-02-26 21:20:03 -0800 Thanks! I can reproduce with Safari/WebKit 4.0.4, but this appears fixed with recent WebKit builds (I tried r55339). 194464 7 paul 2010-02-26 23:06:35 -0800 For reference, I'm pretty sure this was fixed in r46444. 194467 8 ap 2010-02-26 23:34:38 -0800 Maybe we need to land this test case then - r46444 didn't include tests. Re-opening for that. 194555 9 49692 ap 2010-02-27 21:29:45 -0800 Created attachment 49692 proposed patch 194894 10 ap 2010-03-01 18:25:27 -0800 Test committed <http://trac.webkit.org/changeset/55396>. 49673 2010-02-26 20:32:30 -0800 2010-02-27 21:29:45 -0800 Testcase for the main page (requires test-prototype-iframe.html as well) test-prototype.html text/html 241 paul CjxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0Ij4KCkhUTUxJbnB1dEVsZW1lbnQucHJvdG90 eXBlLmZvbyA9ICdmb28nOwoKPC9zY3JpcHQ+Cgo8aWZyYW1lIHNyYz0idGVzdC1wcm90b3R5cGUt aWZyYW1lLmh0bWwiPjwvaWZyYW1lPgoKPGZvcm0gb25zdWJtaXQ9ImFsZXJ0KHRoaXMuZWxlbWVu dHNbMF0uZm9vKTsgcmV0dXJuIGZhbHNlIj4KPGlucHV0IHR5cGU9InN1Ym1pdCIgbmFtZT0iZmRz YSI+CjwvZm9ybT4KCg== 49674 2010-02-26 20:33:16 -0800 2010-02-27 21:29:45 -0800 Testcase for the iframe (is loaded by the first testcase page) test-prototype-iframe.html text/html 219 paul CjxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0Ij4KCkhUTUxGb3JtRWxlbWVudC5wcm90b3R5 cGUuYmFyID0gJ2Jhcic7Cgp3aW5kb3cub25sb2FkID0gZnVuY3Rpb24gKGUpIHsKCXRvcC5kb2N1 bWVudC5ib2R5LmFkZEV2ZW50TGlzdGVuZXIoJ21vdXNlZG93bicsIGZ1bmN0aW9uIChlKSB7CgkJ dmFyIHRhcmdldCA9IGUudGFyZ2V0OwoJfSwgdHJ1ZSk7Cn0KCjwvc2NyaXB0PgoK 49692 2010-02-27 21:29:45 -0800 2010-03-01 13:33:05 -0800 proposed patch AddPrototypeTest.txt text/plain 3620 ap SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDU1MzU2KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTAtMDItMjcgIEFsZXhleSBQcm9za3VyeWFr b3YgIDxhcEBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku CisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI2NTIw CisgICAgICAgIEFjY2Vzc2luZyBhIERPTSBub2RlIGZyb20gYSBwYXJlbnQgd2luZG93IGluIGEg Y2hpbGQgaWZyYW1lIG1vdmVzIHRoZSBwcm90b3R5cGUgdG8gdGhlIGNoaWxkCisgICAgICAgIGlm cmFtZSBhbmQgcmVzZXRzIGl0IGluIHRoZSBwYXJlbnQgd2luZG93CisKKyAgICAgICAgQWRkaW5n IGEgdGVzdCBmb3IgYSBidWcgdGhhdCBubyBsb25nZXIgb2NjdXJzIGluIFRvVC4KKworICAgICAg ICAqIGZhc3QvZG9tL2Nyb3NzLWZyYW1lLW5vZGUtcHJvdG90eXBlLWV4cGVjdGVkLnR4dDogQWRk ZWQuCisgICAgICAgICogZmFzdC9kb20vY3Jvc3MtZnJhbWUtbm9kZS1wcm90b3R5cGUuaHRtbDog QWRkZWQuCisgICAgICAgICogZmFzdC9kb20vcmVzb3VyY2VzL2Nyb3NzLWZyYW1lLW5vZGUtcHJv dG90eXBlLWlmcmFtZS5odG1sOiBBZGRlZC4KKwogMjAxMC0wMi0yNyAgWGFuIExvcGV6ICA8eGxv cGV6QGlnYWxpYS5jb20+CiAKICAgICAgICAgUnViYmVyLXN0YW1wZWQgYnkgR3VzdGF2byBOb3Jv bmhhLgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9kb20vY3Jvc3MtZnJhbWUtbm9kZS1wcm90b3R5 cGUtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvZG9tL2Nyb3Nz LWZyYW1lLW5vZGUtcHJvdG90eXBlLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKKysrIExheW91 dFRlc3RzL2Zhc3QvZG9tL2Nyb3NzLWZyYW1lLW5vZGUtcHJvdG90eXBlLWV4cGVjdGVkLnR4dAko cmV2aXNpb24gMCkKQEAgLTAsMCArMSw1IEBACitUZXN0IGZvciBidWcgMjY1MjA6IEFjY2Vzc2lu ZyBhIERPTSBub2RlIGZyb20gYSBwYXJlbnQgd2luZG93IGluIGEgY2hpbGQgaWZyYW1lIG1vdmVz IHRoZSBwcm90b3R5cGUgdG8gdGhlIGNoaWxkIGlmcmFtZSBhbmQgcmVzZXRzIGl0IGluIHRoZSBw YXJlbnQgd2luZG93LgorCisKKworUEFTUwoKUHJvcGVydHkgY2hhbmdlcyBvbjogTGF5b3V0VGVz dHMvZmFzdC9kb20vY3Jvc3MtZnJhbWUtbm9kZS1wcm90b3R5cGUtZXhwZWN0ZWQudHh0Cl9fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KQWRkZWQ6IHN2bjptaW1lLXR5cGUKICAgKyB0ZXh0L3BsYWluCkFkZGVkOiBzdm46ZW9s LXN0eWxlCiAgICsgbmF0aXZlCgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9kb20vY3Jvc3MtZnJh bWUtbm9kZS1wcm90b3R5cGUuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2Rv bS9jcm9zcy1mcmFtZS1ub2RlLXByb3RvdHlwZS5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0 VGVzdHMvZmFzdC9kb20vY3Jvc3MtZnJhbWUtbm9kZS1wcm90b3R5cGUuaHRtbAkocmV2aXNpb24g MCkKQEAgLTAsMCArMSwyMSBAQAorPHA+VGVzdCBmb3IgPGEgaHJlZj0iaHR0cHM6Ly9idWdzLndl YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI2NTIwIj5idWcgMjY1MjA8L2E+OgorQWNjZXNzaW5n IGEgRE9NIG5vZGUgZnJvbSBhIHBhcmVudCB3aW5kb3cgaW4gYSBjaGlsZCBpZnJhbWUgbW92ZXMg dGhlIHByb3RvdHlwZSB0byB0aGUgY2hpbGQgaWZyYW1lIGFuZCByZXNldHMgaXQgaW4gdGhlIHBh cmVudCB3aW5kb3cuPC9wPgorPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgoraWYgKHdp bmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1w QXNUZXh0KCk7CisKK0hUTUxJbnB1dEVsZW1lbnQucHJvdG90eXBlLmZvbyA9ICdmb28nOworCitm dW5jdGlvbiB0ZXN0KHZhbHVlKQoreworICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJyZXN1 bHQiKS5pbm5lckhUTUwgPSAodmFsdWUgPT0gImZvbyIpID8gIlBBU1MiIDogKCJGQUlMOiAiICsg dmFsdWUpOworfQorCis8L3NjcmlwdD4KKworPGlmcmFtZSBzcmM9InJlc291cmNlcy9jcm9zcy1m cmFtZS1ub2RlLXByb3RvdHlwZS1pZnJhbWUuaHRtbCI+PC9pZnJhbWU+CisKKzxmb3JtIG9uc3Vi bWl0PSJ0ZXN0KHRoaXMuZWxlbWVudHNbMF0uZm9vKTsgcmV0dXJuIGZhbHNlIj4KKzxpbnB1dCB0 eXBlPSJzdWJtaXQiIG5hbWU9ImZkc2EiPgorPGRpdiBpZD1yZXN1bHQ+RkFJTDogVGVzdCBkaWRu J3QgcnVuLiBQbGVhc2UgY2xpY2sgb24gdGhlIGJ1dHRvbiBhYm92ZS48L2Rpdj4KKzwvZm9ybT4K ClByb3BlcnR5IGNoYW5nZXMgb246IExheW91dFRlc3RzL2Zhc3QvZG9tL2Nyb3NzLWZyYW1lLW5v ZGUtcHJvdG90eXBlLmh0bWwKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwpBZGRlZDogc3ZuOm1pbWUtdHlwZQogICArIHRl eHQvaHRtbAoKSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvZG9tL3Jlc291cmNlcy9jcm9zcy1mcmFt ZS1ub2RlLXByb3RvdHlwZS1pZnJhbWUuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9m YXN0L2RvbS9yZXNvdXJjZXMvY3Jvc3MtZnJhbWUtbm9kZS1wcm90b3R5cGUtaWZyYW1lLmh0bWwJ KHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2RvbS9yZXNvdXJjZXMvY3Jvc3MtZnJh bWUtbm9kZS1wcm90b3R5cGUtaWZyYW1lLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTQg QEAKKworPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgorCitIVE1MRm9ybUVsZW1lbnQu cHJvdG90eXBlLmJhciA9ICdiYXInOworCit3aW5kb3cub25sb2FkID0gZnVuY3Rpb24gKGUpIHsK Kwl0b3AuZG9jdW1lbnQuYm9keS5hZGRFdmVudExpc3RlbmVyKCdjbGljaycsIGZ1bmN0aW9uIChl KSB7CisJCXZhciB0YXJnZXQgPSBlLnRhcmdldDsKKwl9LCB0cnVlKTsKKwl0b3AuZG9jdW1lbnQu Z2V0RWxlbWVudHNCeVRhZ05hbWUoImlucHV0IilbMF0uY2xpY2soKTsKK30KKworPC9zY3JpcHQ+ CisKClByb3BlcnR5IGNoYW5nZXMgb246IExheW91dFRlc3RzL2Zhc3QvZG9tL3Jlc291cmNlcy9j cm9zcy1mcmFtZS1ub2RlLXByb3RvdHlwZS1pZnJhbWUuaHRtbApfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkFkZGVkOiBz dm46bWltZS10eXBlCiAgICsgdGV4dC9odG1sCgo=