49123 2010-11-05 21:08:18 -0700 REGRESSION: beforeload events are fired by <object> elements with YouTube embedding code 2010-11-08 17:45:51 -0800 1 1 1 Unclassified WebKit Plug-ins 528+ (Nightly build) Mac (Intel) OS X 10.6 RESOLVED DUPLICATE 44575 Regression P2 Normal --- 1 marc.hoyois webkit-unassigned aestes ap oldest_to_newest 305716 0 marc.hoyois 2010-11-05 21:08:18 -0700 Overview: TWO beforeload events are fired by <object> elements instead of ZERO when their fallback content is used. Steps to Reproduce: Open the attached HTML file. It contains an instance of YouTube's standard embedding code, and alerts for beforeload events. Expected result: The <object> tag is ignored because it has neither the type nor the data attribute. The enclosed <embed> tag is used and fires a beforeload event. Total events fired: 1. Actual result: The <object> tag is ignored because it has neither the type nor the data attribute. The enclosed <embed> tag is used and fires a beforeload event. BUT, the unused <object> tag also fires two beforeload events. Total events fired: 3. Tested in OS X 10.6.4 with WebkKit 6533.18.5, r71367. Notes: The same happens with any other kind of fallback content (<video>, <img>, ...) 305717 1 73162 marc.hoyois 2010-11-05 21:09:29 -0700 Created attachment 73162 An HTML file that reproduces the bug 305731 2 ap 2010-11-05 22:31:41 -0700 Note that I'm getting entirely different behavior with Safari 5.0.2: the movie apparently just loads from the object element, and there is just one beforeload event. With a debug build: ASSERTION FAILED: !m_inBeforeLoadEventHandler (/Users/ap/Safari/OpenSource/WebCore/html/HTMLObjectElement.cpp:282 virtual void WebCore::HTMLObjectElement::updateWidget(bool)) 305740 3 marc.hoyois 2010-11-05 22:51:10 -0700 Yeah I forgot to mention that. Safari 5.0.2 and earlier treat the <object>/<embed> combination in a special way, but that was fixed a few weeks ago in the nightlies (now the <embed> is just like any other fallback). But the problem is already there for other kind of fallback content. I'm attaching an example with <video> fallback. 305741 4 73166 marc.hoyois 2010-11-05 22:52:00 -0700 Created attachment 73166 Another example with <video> fallback 306519 5 aestes 2010-11-08 17:45:51 -0800 I believe this is a dupe of https://bugs.webkit.org/show_bug.cgi?id=44575, although this technically is documenting two separate issues: 1) beforeload is fired twice on object tags (dupe of 44575). 2) beforeload is fired on object tags that will never fire a load event (new bug AFAIK). For #2, I'm not sure whether its correct to fire a beforeload event for objects with fallback content that that we know we won't load. That warrants its own discussion that is separate from #1, IMO. *** This bug has been marked as a duplicate of bug 44575 *** 73162 2010-11-05 21:09:29 -0700 2010-11-05 21:09:29 -0700 An HTML file that reproduces the bug test.html text/html 660 marc.hoyois PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPHNjcmlwdD4KICAgIGZ1bmN0aW9uIGhhbmRs ZUV2ZW50KGV2ZW50KSB7CiAgICAgICAgYWxlcnQoZXZlbnQudGFyZ2V0Lm5vZGVOYW1lICsgIlxu IiArIGV2ZW50LnVybCk7CiAgICB9CgogICAgZG9jdW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigiYmVm b3JlbG9hZCIsIGhhbmRsZUV2ZW50LCB0cnVlKTsKPC9zY3JpcHQ+CjwvaGVhZD4KPGJvZHk+CiAg ICA8b2JqZWN0IHdpZHRoPSI2NDAiIGhlaWdodD0iMzg1Ij48cGFyYW0gbmFtZT0ibW92aWUiIHZh bHVlPSJodHRwOi8vd3d3LnlvdXR1YmUuY29tL3YvS3RwYXR4cmZ0eDg/ZnM9MSI+PC9wYXJhbT48 cGFyYW0gbmFtZT0iYWxsb3dGdWxsU2NyZWVuIiB2YWx1ZT0idHJ1ZSI+PC9wYXJhbT48cGFyYW0g bmFtZT0iYWxsb3dzY3JpcHRhY2Nlc3MiIHZhbHVlPSJhbHdheXMiPjwvcGFyYW0+PGVtYmVkIHNy Yz0iaHR0cDovL3d3dy55b3V0dWJlLmNvbS92L0t0cGF0eHJmdHg4P2ZzPTEmYW1wO2hsPWVuX1VT IiB0eXBlPSJhcHBsaWNhdGlvbi94LXNob2Nrd2F2ZS1mbGFzaCIgYWxsb3dzY3JpcHRhY2Nlc3M9 ImFsd2F5cyIgYWxsb3dmdWxsc2NyZWVuPSJ0cnVlIiB3aWR0aD0iNjQwIiBoZWlnaHQ9IjM4NSI+ PC9lbWJlZD48L29iamVjdD4KPC9ib2R5Pgo8L2h0bWw+ 73166 2010-11-05 22:52:00 -0700 2010-11-05 22:52:00 -0700 Another example with <video> fallback test2.html text/html 311 marc.hoyois PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPHNjcmlwdD4KICAgIGZ1bmN0aW9uIGhhbmRs ZUV2ZW50KGV2ZW50KSB7CiAgICAgICAgYWxlcnQoZXZlbnQudGFyZ2V0Lm5vZGVOYW1lICsgIlxu IiArIGV2ZW50LnVybCk7CiAgICB9CgogICAgZG9jdW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigiYmVm b3JlbG9hZCIsIGhhbmRsZUV2ZW50LCB0cnVlKTsKPC9zY3JpcHQ+CjwvaGVhZD4KPGJvZHk+CiAg ICA8b2JqZWN0IHdpZHRoPSI2NDAiIGhlaWdodD0iMzg1Ij48dmlkZW8gc3JjPSJkdW1teS5tcDQi Pjwvb2JqZWN0Pgo8L2JvZHk+CjwvaHRtbD4=