15484 2007-10-12 18:30:58 -0700 Safari (and opera) fire two onload events while FireFox fires only one 2022-09-20 13:46:20 -0700 1 1 1 Unclassified WebKit Page Loading 523.x (Safari 3) Mac OS X 10.4 RESOLVED CONFIGURATION CHANGED https://bugs.webkit.org/show_bug.cgi?id=219469 InRadar, NeedsReduction P2 Normal --- 1 eric webkit-unassigned achristensen ahmad.saleem792 ap beidson bolinfest cdumez dewei_zhu slewis webkit-bug-importer oldest_to_newest 58392 0 eric 2007-10-12 18:30:58 -0700 Safari (and opera) fire two onload events while FireFox fires only one I'm not sure who's correct, but I'm going to lean towards the browser with the larger market share. The "bug" here is exposed by the ordering of the insertion of the iframe into the document and the setting of the url. FireFox only fires the onload event once, after the final URL has been set. Opera and Safari fire the onload handler right when the iframe is inserted, and then again after the src value has been changed. I tried at least once to reduce this and seen the bug go away. I expect it's partially timing related. 58393 1 16647 eric 2007-10-12 18:31:16 -0700 Created attachment 16647 test case 58394 2 16648 eric 2007-10-12 18:34:19 -0700 Created attachment 16648 test case 58395 3 andrew 2007-10-12 18:43:43 -0700 The output in IE6 and IE7 is: about to set URL: resources/viewspec-parser.svg Starting: resources/viewspec-parser.svg Only one iframe is loaded. Seems the test needs to be modified to work properly in IE. 318006 4 bolinfest 2010-12-06 18:40:01 -0800 Test page: http://bolinfest.com/webkit/iframe_onload.html I would argue that the first onload event fired by Webkit is the problematic one: the body.onload in the iframe should fire first, then the iframe.onload (as it does in Firefox). 1872668 5 ahmad.saleem792 2022-05-29 17:02:46 -0700 I am still able to reproduce the following bug in Safari 15.5 on macOS 12.4. Chrome Canary 104 and Firefox Nightly 102 both - fires "iframe loaded" dialog just once after "Hello World". Test case - http://bolinfest.com/webkit/iframe_onload.html 1872997 6 webkit-bug-importer 2022-05-31 09:53:52 -0700 <rdar://problem/94158603> 1899926 7 ahmad.saleem792 2022-09-20 13:46:20 -0700 Unable to reproduce this in Safari Technology Preview 153 and it just fire "Hello World" and after it, it just load "iframe loaded" dialog once similar to other browsers. I am marking this as "RESOLVED CONFIGURATION CHANGED". Please reopen, if it is reproducible. Thanks! Appreciate if someone can add related Commit details for future reference. 16647 2007-10-12 18:31:16 -0700 2007-10-12 18:34:19 -0700 test case viewspec-parser.html text/html 3741 eric PGh0bWw+CjxoZWFkPgo8bGluayByZWw9InN0eWxlc2hlZXQiIGhyZWY9Ii4uLy4uL2Zhc3QvanMv cmVzb3VyY2VzL2pzLXRlc3Qtc3R5bGUuY3NzIj4KPHNjcmlwdCBzcmM9Ii4uLy4uL2Zhc3QvanMv cmVzb3VyY2VzL2pzLXRlc3QtcHJlLmpzIj48L3NjcmlwdD4KPHNjcmlwdCBzcmM9InJlc291cmNl cy9zY3JpcHRlZC1yYW5kb20uanMiPjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5Pgo8cCBpZD0iZGVz Y3JpcHRpb24iPjwvcD4KPGRpdiBpZD0iY29uc29sZSI+PC9kaXY+CjxzY3JpcHQ+CiAgICBkZXNj cmlwdGlvbigiVGhpcyB0ZXN0IGZ1enplcyB0aGUgdHJhbnNmb3JtIHBhcnNlciB3aXRoIHNlbWkt cmFuZG9tIGF0dHJpYnV0ZSB2YWx1ZXMgYW5kIGR1bXBzIHRoZSByZXN1bHRzIG9mIGFueSB2YWx1 ZXMgdGhhdCBwYXJzZSBzdWNjZXNzZnVsbHkuIik7CgogICAgdmFyIGF0dHJpYnV0ZXMgPSBbIC8v IGxpc3Qgb2Ygdmlld3NwZWMgYXR0cmlidXRlIG5hbWUgYW5kIG1pbmltdW0gYXJndW1lbnQgY291 bnQKICAgICAgICBbJ3ZpZXdCb3gnLCA0XSwKICAgICAgICBbJ3ByZXNlcnZlQXNwZWN0UmF0aW8n LCA0XSwKICAgICAgICBbJ3RyYW5zZm9ybScsIDRdLAogICAgICAgIFsnem9vbUFuZFBhbicsIDRd LAogICAgICAgIFsndmlld1RhcmdldCcsIDRdCiAgICBdOwogICAgCiAgICB2YXIgcHJlc2VydmVB c3BlY3RSYXRpb1ZhbHVlcyA9IFsgIk1pbiIsICJNYXgiLCAiTWlkIiBdOwogICAgCiAgICB2YXIg Y2hhcmFjdGVycyA9IFsKICAgICAgICAiMCIsCiAgICAgICAgIjEiLAogICAgICAgICIyIiwKICAg ICAgICAiMyIsCiAgICAgICAgIjQiLAogICAgICAgICI1IiwKICAgICAgICAiNiIsCiAgICAgICAg IjciLAogICAgICAgICI4IiwKICAgICAgICAiOSIsCiAgICAgICAgIi4iLAogICAgICAgICJlIiwK ICAgICAgICAiKyIsCiAgICAgICAgIi0iLAogICAgICAgICJlIiwKICAgICAgICAiKCIsCiAgICAg ICAgIikiLAogICAgICAgICIgIiwgLy8gbm90IGEgdmFsaWQgZnJhZ21lbnQgY2hhcgogICAgICAg ICJcdCIsIC8vIG5vdCBhIHZhbGlkIGZyYWdtZW50IGNoYXIKICAgICAgICAiLCIKICAgIF07CiAg ICAKICAgIGZ1bmN0aW9uIHZpZXdTcGVjVG9TdHJpbmcodmlld1NwZWMpCiAgICB7CiAgICAgICAg aWYgKCF2aWV3U3BlYykKICAgICAgICAgIHJldHVybiB2aWV3U3BlYzsKICAgICAgICAKICAgICAg ICB2YXIgYXR0cmlidXRlcyA9IFtdOwogICAgICAgIGlmICh2aWV3U3BlYy50cmFuc2Zvcm0pCiAg ICAgICAgICBhdHRyaWJ1dGVzLnB1c2goInRyYW5zZm9ybSgiICsgdmlld1NwZWMudHJhbnNmb3Jt ICsgIikiKTsKICAgICAgICBpZiAodmlld1NwZWMudmlld0JveCkKICAgICAgICAgIGF0dHJpYnV0 ZXMucHVzaCgidmlld0JveCgiICsgdmlld1NwZWMudmlld0JveCArICIpIik7CiAgICAgICAgaWYg KHZpZXdTcGVjLnByZXNlcnZlQXNwZWN0UmF0aW8pCiAgICAgICAgICBhdHRyaWJ1dGVzLnB1c2go InByZXNlcnZlQXNwZWN0UmF0aW8oIiArIHZpZXdTcGVjLnByZXNlcnZlQXNwZWN0UmF0aW8gKyAi KSIpOwogICAgICAgIGlmICh2aWV3U3BlYy56b29tQW5kUGFuKQogICAgICAgICAgYXR0cmlidXRl cy5wdXNoKCJ6b29tQW5kUGFuKCIgKyB2aWV3U3BlYy56b29tQW5kUGFuICsgIikiKTsKICAgICAg ICBpZiAodmlld1NwZWMudmlld1RhcmdldCkKICAgICAgICAgIGF0dHJpYnV0ZXMucHVzaCgidmll d1RhcmdldCgiICsgdmlld1NwYWMudmlld1RhcmdldCArICIpIik7CiAgICAgICAgICAKICAgICAg ICByZXR1cm4gInN2Z1ZpZXcoIiArIGF0dHJpYnV0ZXMuam9pbigiOyIpICsgIikiOwogICAgfQog ICAgCiAgICB2YXIgdGVzdE51bWJlciA9IDA7CiAgICB2YXIgdGVzdFN0cmluZyA9ICJbaW5pdGlh bCB2aWV3XSIKCiAgICBmdW5jdGlvbiB0ZXN0RnJhZ21lbnQoc3RyaW5nKSB7CiAgICAgIHZhciBv bGRFbWJlZCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJvYmplY3QiKTsKICAgICAgaWYgKG9s ZEVtYmVkKSB7CiAgICAgICAgb2xkRW1iZWQuaWQ9ImZvbyI7CiAgICAgIH0KICAgICAgdmFyIGVt YmVkRWxlbWVudCA9IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoImlmcmFtZSIpOwogICAgICBlbWJl ZEVsZW1lbnQuc2V0QXR0cmlidXRlKCJpZCIsICJvYmplY3QiKTsKICAgICAgZW1iZWRFbGVtZW50 LnNldEF0dHJpYnV0ZSgid2lkdGgiLCAiMTAwIik7CiAgICAgIGVtYmVkRWxlbWVudC5zZXRBdHRy aWJ1dGUoImhlaWdodCIsICIxMDAiKTsKICAgICAgZW1iZWRFbGVtZW50LnNldEF0dHJpYnV0ZSgi b25sb2FkIiwgImNvbnRpbnVlRnV6emluZyhldmVudCkiKTsKICAgICAgZG9jdW1lbnQuYm9keS5h cHBlbmRDaGlsZChlbWJlZEVsZW1lbnQpOwogICAgICB2YXIgbmV3VVJMID0gInJlc291cmNlcy92 aWV3c3BlYy1wYXJzZXIuc3ZnIiArIHN0cmluZzsKICAgICAgZGVidWcoImFib3V0IHRvIHNldCBV Ukw6ICIgKyBuZXdVUkwpCiAgICAgIGVtYmVkRWxlbWVudC5zcmMgPSBuZXdVUkw7CiAgICAgIGRl YnVnKCJTdGFydGluZzogIiArIG5ld1VSTCk7CiAgICB9CgogICAgZnVuY3Rpb24gc3RhcnROZXh0 VGVzdCgpCiAgICB7CiAgICAgIGRlYnVnKCJpbnRpbWVyIik7CiAgICAgIHRlc3RGcmFnbWVudCgi Iik7CiAgICAgIGRlYnVnKCJvdXR0aW1lciIpOwogICAgfQoKICAgIGZ1bmN0aW9uIGNvbnRpbnVl RnV6emluZyhldmVudCkKICAgIHsKICAgICAgZGVidWcoImJlZ2luICIgKyB0ZXN0TnVtYmVyICsg IiAiICsgZXZlbnQudGFyZ2V0KTsKICAgICAgdGVzdE51bWJlcisrOwogICAgICAKICAgICAgICB2 YXIgZW1iZWRFbGVtZW50ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoIm9iamVjdCIpOwogICAg ICAgIGlmIChlbWJlZEVsZW1lbnQgJiYgZW1iZWRFbGVtZW50LmNvbnRlbnREb2N1bWVudCkKICAg ICAgICAgIGRlYnVnKCJQYXJzZWQ6ICIgKyB2aWV3U3BlY1RvU3RyaW5nKGVtYmVkRWxlbWVudC5j b250ZW50RG9jdW1lbnQuY3VycmVudFZpZXcpICsgIiBmcm9tOiAiICsgdGVzdFN0cmluZyk7CiAg ICAgICAgZWxzZQogICAgICAgICAgZGVidWcoIm5vIHN2Z2RvY3VtZW50Iik7CiAgICAgICAgCiAg ICAgICAgaWYgKHRlc3ROdW1iZXIgPT0gMCkKICAgICAgICAgIHRlc3RTdHJpbmcgPSAic3ZnVmll dyI7CiAgICAgICAgZWxzZSBpZiAodGVzdE51bWJlciA9PSAxKQogICAgICAgICAgdGVzdFN0cmlu ZyA9ICJzdmdWaWV3KCI7CiAgICAgICAgZWxzZSBpZiAodGVzdE51bWJlciA9PSAyKQogICAgICAg ICAgdGVzdFN0cmluZyA9ICJzdmdWaWV3KCIgKyBTdHJpbmcuZnJvbUNoYXJDb2RlKDApOwogICAg ICAgIGVsc2UgaWYgKHRlc3ROdW1iZXIgPT0gMykKICAgICAgICAgIHRlc3RTdHJpbmcgPSAic3Zn VmlldygiICsgU3RyaW5nLmZyb21DaGFyQ29kZSgwKSArICIpIjsKICAgICAgICBlbHNlIHsKICAg ICAgICAgIGRlYnVnKCc8YnI+PHNwYW4gY2xhc3M9InBhc3MiPlRFU1QgQ09NUExFVEU8L3NwYW4+ Jyk7CiAgICAgICAgICByZXR1cm47CiAgICAgICAgfQogICAgICAgIC8vIHRoaXMgbGV0cyB1cyBv dXQgb2YgdGhlIG9ubG9hZCBoYW5kbGVyIHNvIHdlIGRvbid0IG92ZXJydW4gdGhlIHN0YWNrCiAg ICAgICAgZGVidWcoInRpbWVyIik7CiAgICAgICAgCiAgICAgICAgd2luZG93LnNldFRpbWVvdXQo c3RhcnROZXh0VGVzdCwgMjAwMCk7CiAgICAgICAgZGVidWcoImRvbmUgdGltZXIiKTsKICAgIH0K dGVzdEZyYWdtZW50KCIiKTsKPC9zY3JpcHQ+CjwvaHRtbD4K 16648 2007-10-12 18:34:19 -0700 2007-10-12 18:34:19 -0700 test case viewspec-parser.html text/html 1583 eric PGh0bWw+Cjxib2R5Pgo8cCBpZD0iZGVzY3JpcHRpb24iPjwvcD4KPGRpdiBpZD0iY29uc29sZSI+ PC9kaXY+CjxzY3JpcHQ+CiAgICBmdW5jdGlvbiBkZWJ1ZyhzKSB7CiAgICAgIHZhciBsb2dEaXYg PSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiY29uc29sZSIpOwogICAgICBsb2dEaXYuYXBwZW5k Q2hpbGQoZG9jdW1lbnQuY3JlYXRlVGV4dE5vZGUocykpOwogICAgICBsb2dEaXYuYXBwZW5kQ2hp bGQoZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgiYnIiKSk7CiAgICB9CiAgICAKICAgIHZhciB0ZXN0 TnVtYmVyID0gMDsKICAgIHZhciB0ZXN0U3RyaW5nID0gIltpbml0aWFsIHZpZXddIgoKICAgIGZ1 bmN0aW9uIHRlc3RGcmFnbWVudChzdHJpbmcpIHsKICAgICAgdmFyIG9sZEVtYmVkID0gZG9jdW1l bnQuZ2V0RWxlbWVudEJ5SWQoIm9iamVjdCIpOwogICAgICBpZiAob2xkRW1iZWQpIHsKICAgICAg ICBvbGRFbWJlZC5pZD0iZm9vIjsKICAgICAgfQogICAgICB2YXIgZW1iZWRFbGVtZW50ID0gZG9j dW1lbnQuY3JlYXRlRWxlbWVudCgiaWZyYW1lIik7CiAgICAgIGVtYmVkRWxlbWVudC5zZXRBdHRy aWJ1dGUoImlkIiwgIm9iamVjdCIpOwogICAgICBlbWJlZEVsZW1lbnQuc2V0QXR0cmlidXRlKCJ3 aWR0aCIsICIxMDAiKTsKICAgICAgZW1iZWRFbGVtZW50LnNldEF0dHJpYnV0ZSgiaGVpZ2h0Iiwg IjEwMCIpOwogICAgICBlbWJlZEVsZW1lbnQuc2V0QXR0cmlidXRlKCJvbmxvYWQiLCAiY29udGlu dWVGdXp6aW5nKGV2ZW50KSIpOwogICAgICBkb2N1bWVudC5ib2R5LmFwcGVuZENoaWxkKGVtYmVk RWxlbWVudCk7CiAgICAgIHZhciBuZXdVUkwgPSAicmVzb3VyY2VzL3ZpZXdzcGVjLXBhcnNlci5z dmciICsgc3RyaW5nOwogICAgICBkZWJ1ZygiYWJvdXQgdG8gc2V0IFVSTDogIiArIG5ld1VSTCkK ICAgICAgZW1iZWRFbGVtZW50LnNyYyA9IG5ld1VSTDsKICAgICAgZGVidWcoIlN0YXJ0aW5nOiAi ICsgbmV3VVJMKTsKICAgIH0KCiAgICBmdW5jdGlvbiBzdGFydE5leHRUZXN0KCkKICAgIHsKICAg ICAgZGVidWcoImludGltZXIiKTsKICAgICAgdGVzdEZyYWdtZW50KCIiKTsKICAgICAgZGVidWco Im91dHRpbWVyIik7CiAgICB9CgogICAgZnVuY3Rpb24gY29udGludWVGdXp6aW5nKGV2ZW50KQog ICAgewogICAgICBkZWJ1ZygiYmVnaW4gIiArIHRlc3ROdW1iZXIgKyAiICIgKyBldmVudC50YXJn ZXQpOwogICAgICB0ZXN0TnVtYmVyKys7CiAgICAgIAogICAgICAgIGlmICh0ZXN0TnVtYmVyID4g MikgewogICAgICAgICAgZGVidWcoJzxicj48c3BhbiBjbGFzcz0icGFzcyI+VEVTVCBDT01QTEVU RTwvc3Bhbj4nKTsKICAgICAgICAgIHJldHVybjsKICAgICAgICB9CiAgICAgICAgLy8gdGhpcyBs ZXRzIHVzIG91dCBvZiB0aGUgb25sb2FkIGhhbmRsZXIgc28gd2UgZG9uJ3Qgb3ZlcnJ1biB0aGUg c3RhY2sKICAgICAgICBkZWJ1ZygidGltZXIiKTsKICAgICAgICAKICAgICAgICB3aW5kb3cuc2V0 VGltZW91dChzdGFydE5leHRUZXN0LCAyMDAwKTsKICAgICAgICBkZWJ1ZygiZG9uZSB0aW1lciIp OwogICAgfQp0ZXN0RnJhZ21lbnQoIiIpOwo8L3NjcmlwdD4KPC9odG1sPgo=