24564 2009-03-12 15:23:11 -0700 Add test to ensure that the JS document wrapper is created in the correct context 2009-03-13 09:21:00 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 pam pam oldest_to_newest 113510 0 pam 2009-03-12 15:23:11 -0700 Once upon a time, Chromium had a bug in which the JS wrapper for the document in an inner frame was created using the prototype chain from the outer frame if the document was accessed from the outer frame first. Add a test to catch that. 113511 1 28560 pam 2009-03-12 15:25:37 -0700 Created attachment 28560 New test + result 113570 2 28560 ap 2009-03-13 03:37:52 -0700 Comment on attachment 28560 New test + result r=me I think that the test could be slightly cleaner if it used iframe onload attribute, rather than relying on some script run in the subframe. But maybe that wouldn't trigger the bug for some reason? 113598 3 pam 2009-03-13 09:21:00 -0700 Hm... I'm not certain either. I'll go ahead and land this as it is for now, and ask the creator of the test to further improve it if he can. (It started out using timeouts and polling rather than cross-frame function calls.) Landed in r41668. 28560 2009-03-12 15:25:37 -0700 2009-03-13 03:37:52 -0700 New test + result 24564a.txt text/plain 3012 pam SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDQxNjU1 KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMDktMDMt MTIgIFBhbWVsYSBHcmVlbmUgIDxwYW1AY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2Vk IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFkZCBhIHRlc3QgdG8gY2hlY2sgdGhhdCB0 aGUgSlMgd3JhcHBlciBmb3IgdGhlIGRvY3VtZW50IGluIGFuCisgICAgICAgIGlubmVyIGZyYW1l IGlzIGNyZWF0ZWQgaW4gdGhlIGNvcnJlY3QgY29udGV4dCB3aGVuIGl0J3MgYWNjZXNzZWQKKyAg ICAgICAgZnJvbSB0aGUgb3V0ZXIgZnJhbWUgZmlyc3QuCisKKyAgICAgICAgaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0NTY0CisKKyAgICAgICAgKiBmYXN0L2RvbS9y ZXNvdXJjZXMvd3JhcHBlci1jb250ZXh0LWlubmVyLmh0bWw6IEFkZGVkLgorICAgICAgICAqIGZh c3QvZG9tL3dyYXBwZXItY29udGV4dC1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIGZh c3QvZG9tL3dyYXBwZXItY29udGV4dC5odG1sOiBBZGRlZC4KKwogMjAwOS0wMy0xMiAgQWFyb24g Qm9vZG1hbiAgPGFhQGNocm9taXVtLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXZlIEh5 YXR0LgpJbmRleDogZmFzdC9kb20vd3JhcHBlci1jb250ZXh0LWV4cGVjdGVkLnR4dAo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBmYXN0L2RvbS93cmFwcGVyLWNvbnRleHQtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAw KQorKysgZmFzdC9kb20vd3JhcHBlci1jb250ZXh0LWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkK QEAgLTAsMCArMSw2IEBACitUZXN0cyB0aGF0IG5vZGUgd3JhcHBlcnMgYXJlIGNyZWF0ZWQgaW4g dGhlIGNvbnRleHQgdG8gd2hpY2ggdGhleSBiZWxvbmcuIFlvdSBzaG91bGQgc2VlIFBBU1MgYmVs b3cuIAorUnVubmluZy4uLgorCitQQVNTCisKKwpJbmRleDogZmFzdC9kb20vd3JhcHBlci1jb250 ZXh0Lmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQotLS0gZmFzdC9kb20vd3JhcHBlci1jb250ZXh0Lmh0bWwJKHJl dmlzaW9uIDApCisrKyBmYXN0L2RvbS93cmFwcGVyLWNvbnRleHQuaHRtbAkocmV2aXNpb24gMCkK QEAgLTAsMCArMSwzMSBAQAorPGh0bWw+Cis8Ym9keT4KK1Rlc3RzIHRoYXQgbm9kZSB3cmFwcGVy cyBhcmUgY3JlYXRlZCBpbiB0aGUgY29udGV4dCB0byB3aGljaCB0aGV5IGJlbG9uZy4KK1lvdSBz aG91bGQgc2VlIFBBU1MgYmVsb3cuCis8YnI+Cis8ZGl2IGlkPSJvdXRwdXQiPjwvZGl2PgorPGlm cmFtZSBpZD0iaW5uZXIiIHNyYz0icmVzb3VyY2VzL3dyYXBwZXItY29udGV4dC1pbm5lci5odG1s Ij48L2lmcmFtZT4KKzxzY3JpcHQ+CitpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKSB7 CisgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCgpOworICAgIGxheW91dFRlc3RD b250cm9sbGVyLndhaXRVbnRpbERvbmUoKTsKK30KKworZnVuY3Rpb24gd3JpdGVPdXRwdXQocykg eworICAgIHZhciBwYXJhZ3JhcGggPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50KCJwIik7CisgICAg cGFyYWdyYXBoLmlubmVySFRNTCA9IHM7CisgICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoIm91 dHB1dCIpLmFwcGVuZENoaWxkKHBhcmFncmFwaCk7Cit9CisKKy8vIFVzZWQgdG8gY3JlYXRlIHRo ZSBkb2N1bWVudCB3cmFwcGVyLgordmFyIGlubmVyRG9jdW1lbnQ7CisKKy8vIE9uY2UgdGhlIGlu bmVyIGZyYW1lIGhhcyBsb2FkZWQsIHJ1biB0aGUgdGVzdC4KK2Z1bmN0aW9uIGlubmVySGFzTG9h ZGVkKCkgeworICAgIHdyaXRlT3V0cHV0KCJSdW5uaW5nLi4uIik7CisgICAgaW5uZXJEb2N1bWVu dCA9IGlubmVyLmRvY3VtZW50OworICAgIGlubmVyLnJ1blRlc3QoKTsKK30KKzwvc2NyaXB0Pgor PC9ib2R5PgorPC9odG1sPgpJbmRleDogZmFzdC9kb20vcmVzb3VyY2VzL3dyYXBwZXItY29udGV4 dC1pbm5lci5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGZhc3QvZG9tL3Jlc291cmNlcy93cmFwcGVyLWNv bnRleHQtaW5uZXIuaHRtbAkocmV2aXNpb24gMCkKKysrIGZhc3QvZG9tL3Jlc291cmNlcy93cmFw cGVyLWNvbnRleHQtaW5uZXIuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwyNCBAQAorPGh0 bWw+Cis8Ym9keSBvbmxvYWQ9Im5vdGlmeUxvYWRlZCgpIj4KKzxzY3JpcHQ+CitmdW5jdGlvbiBu b3RpZnlMb2FkZWQoKSB7CisgIHRvcC5pbm5lckhhc0xvYWRlZCgpOworfQorCisvLyBBZnRlciB0 aGUgdG9wIGZyYW1lIGhhcyBhY2Nlc3NlZCB0aGUgZG9jdW1lbnQgb2YgdGhpcyBmcmFtZSwgd2UK Ky8vIHRlc3QgdGhhdCB0aGUgZG9jdW1lbnQgd3JhcHBlciB3YXMgY3JlYXRlZCBpbiB0aGlzIGNv bnRleHQgYW5kCisvLyBub3QgaW4gdGhlIHRvcCBjb250ZXh0LgorZnVuY3Rpb24gcnVuVGVzdCgp IHsKKyAgICAgdmFyIHBhcmFncmFwaCA9IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoJ3AnKTsKKyAg ICAgaWYgKEhUTUxFbGVtZW50LnByb3RvdHlwZS5pc1Byb3RvdHlwZU9mKHBhcmFncmFwaCkpIHsK KyAgICAgICAgIHRvcC53cml0ZU91dHB1dCgiUEFTUyIpOworICAgICB9IGVsc2UgeworICAgICAg ICAgdG9wLndyaXRlT3V0cHV0KCJGQUlMIik7CisgICAgIH0KKyAgICAgaWYgKHdpbmRvdy5sYXlv dXRUZXN0Q29udHJvbGxlcikgeworICAgICAgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIubm90aWZ5 RG9uZSgpOworICAgICB9Cit9Cis8L3NjcmlwdD4KKzwvYm9keT4KKzwvaHRtbD4K