27623 2009-07-23 13:27:55 -0700 Video elements fires another "load" event when attached to DOM 2009-07-30 09:06:45 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Mac (Intel) OS X 10.5 RESOLVED FIXED P2 Normal --- 1 charles_ying eric.carlson rik simon.fraser oldest_to_newest 134227 0 33363 charles_ying 2009-07-23 13:27:55 -0700 Created attachment 33363 test reduction When a video element not attached to the DOM is loaded first (prior to attachment), then attached, a second "load" event is fired on the video element. This manifests itself as a repeated load event in the sample code which adds the video to the DOM only after the video has finished loading. This is illustrated by commenting out the "jQuery("#container").append(video);" line from the attached reduction. 135697 1 eric.carlson 2009-07-29 15:39:00 -0700 Looks like you forgot to attach the test case. 135700 2 charles_ying 2009-07-29 15:47:35 -0700 I think I attached it, it appears in the attachments as "test reduction" 135850 3 33784 eric.carlson 2009-07-30 08:30:05 -0700 Created attachment 33784 proposed patch 135858 4 eric.carlson 2009-07-30 09:06:45 -0700 http://trac.webkit.org/changeset/46581 33363 2009-07-23 13:27:55 -0700 2009-07-23 13:27:55 -0700 test reduction video-events.html text/html 651 charles_ying PGh0bWw+CjxoZWFkPgo8L2hlYWQ+Cjxib2R5PgoKPGRpdiBpZD0iY29udGFpbmVyIj48L2Rpdj4K CjxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0IiBzcmM9Imh0dHA6Ly9hamF4Lmdvb2dsZWFw aXMuY29tL2FqYXgvbGlicy9qcXVlcnkvMS4zLjIvanF1ZXJ5Lm1pbi5qcyI+PC9zY3JpcHQ+Cgo8 c2NyaXB0IHR5cGU9InRleHQvamF2YXNjcmlwdCI+CnZhciB1cmwgPSAiaHR0cDovL21vdmllcy5h cHBsZS5jb20vbW92aWVzL3BhcmFtb3VudC9pcm9uX21hbi9pcm9uX21hbi10bHIyX2guNjQwLm1v diI7CgpmdW5jdGlvbiBsb2cobXNnKQp7Cgljb25zb2xlLmxvZyhtc2cpOwp9Cgpsb2coInN0YXJ0 aW5nIHZpZGVvIGxvYWQiKTsKdmFyIHZpZGVvID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgidmlk ZW8iKTsKCnZpZGVvLmFkZEV2ZW50TGlzdGVuZXIoImxvYWQiLCBmdW5jdGlvbiAoZSkKewoJaWYg KGUudGFyZ2V0ICE9IGRvY3VtZW50KQoJewoJCWxvZygibG9hZCBldmVudCB0cmlnZ2VyZWQiKTsK CQlsb2coZSk7CgkJalF1ZXJ5KCIjY29udGFpbmVyIikuYXBwZW5kKHZpZGVvKTsKCQl2aWRlby5w bGF5KCk7Cgl9Cn0sIGZhbHNlKTsKCnZpZGVvLnNyYyA9IHVybDsKdmlkZW8ubG9hZCgpOwoKPC9z Y3JpcHQ+CgoKPC9ib2R5Pgo8L2h0bWw+ 33784 2009-07-30 08:30:05 -0700 2009-07-30 08:49:16 -0700 proposed patch patch_2.txt text/plain 4007 eric.carlson SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0NjU4MCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTYgQEAKKzIwMDktMDctMzAgIEVyaWMgQ2FybHNvbiAgPGVyaWMuY2FybHNvbkBh cHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg ICAgVmlkZW8gZWxlbWVudHMgZmlyZXMgYW5vdGhlciAibG9hZCIgZXZlbnQgd2hlbiBhdHRhY2hl ZCB0byBET00KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk PTI3NjIzCisKKyAgICAgICAgVGVzdDogbWVkaWEvbWVkaWEtbG9hZC1ldmVudC5odG1sCisKKyAg ICAgICAgKiBodG1sL0hUTUxNZWRpYUVsZW1lbnQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6SFRN TE1lZGlhRWxlbWVudDo6aW5zZXJ0ZWRJbnRvRG9jdW1lbnQpOgorICAgICAgICAgICAgT25seSBz Y2hlZHVsZSBsb2FkIHdoZW4gZWxlbWVudCdzIG5ldHdvcmsgc3RhdGUgaXMgTkVUV09SS19FTVBU WS4KKwogMjAwOS0wNy0zMCAgQW50b24gTXVoaW4gIDxhbnRvbm1AY2hyb21pdW0ub3JnPgogCiAg ICAgICAgIFJldmlld2VkIGJ5IERhdmlkIExldmluLgpJbmRleDogV2ViQ29yZS9odG1sL0hUTUxN ZWRpYUVsZW1lbnQuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvaHRtbC9IVE1MTWVkaWFFbGVt ZW50LmNwcAkocmV2aXNpb24gNDY1ODApCisrKyBXZWJDb3JlL2h0bWwvSFRNTE1lZGlhRWxlbWVu dC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTIzNCw3ICsyMzQsNyBAQCBSZW5kZXJPYmplY3QqIEhU TUxNZWRpYUVsZW1lbnQ6OmNyZWF0ZVJlCiB2b2lkIEhUTUxNZWRpYUVsZW1lbnQ6Omluc2VydGVk SW50b0RvY3VtZW50KCkKIHsKICAgICBIVE1MRWxlbWVudDo6aW5zZXJ0ZWRJbnRvRG9jdW1lbnQo KTsKLSAgICBpZiAoIXNyYygpLmlzRW1wdHkoKSkKKyAgICBpZiAoIXNyYygpLmlzRW1wdHkoKSAm JiBtX25ldHdvcmtTdGF0ZSA9PSBORVRXT1JLX0VNUFRZKQogICAgICAgICBzY2hlZHVsZUxvYWQo KTsKIH0KIApJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91 dFRlc3RzL0NoYW5nZUxvZwkocmV2aXNpb24gNDY1ODApCisrKyBMYXlvdXRUZXN0cy9DaGFuZ2VM b2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMyBAQAorMjAwOS0wNy0zMCAgRXJpYyBDYXJs c29uICA8ZXJpYy5jYXJsc29uQGFwcGxlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICBWaWRlbyBlbGVtZW50cyBmaXJlcyBhbm90aGVyICJsb2Fk IiBldmVudCB3aGVuIGF0dGFjaGVkIHRvIERPTQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mjc2MjMKKworICAgICAgICAqIG1lZGlhL21lZGlhLWxvYWQt ZXZlbnQtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBtZWRpYS9tZWRpYS1sb2FkLWV2 ZW50Lmh0bWw6IEFkZGVkLgorCiAyMDA5LTA3LTI5ICBKaWFuIExpICA8amlhbmxpQGNocm9taXVt Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXJpbiBBZGxlci4KSW5kZXg6IExheW91dFRl c3RzL21lZGlhL21lZGlhLWxvYWQtZXZlbnQtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExh eW91dFRlc3RzL21lZGlhL21lZGlhLWxvYWQtZXZlbnQtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAw KQorKysgTGF5b3V0VGVzdHMvbWVkaWEvbWVkaWEtbG9hZC1ldmVudC1leHBlY3RlZC50eHQJKHJl dmlzaW9uIDApCkBAIC0wLDAgKzEsMTggQEAKK1Rlc3QgdGhhdCBtZWRpYSBmaWxlIGlzIG5vdCBy ZWxvYWRlZCB3aGVuIGFuIGVsZW1lbnQgaXMgaW5zZXJ0ZWQgaW50byB0aGUgRE9NLgorCitSVU4o bWVkaWEgPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50KCdhdWRpbycpKQorUlVOKG1lZGlhLnNyYyA9 ICdjb250ZW50L3Rlc3Qud2F2JykKK1JVTihtZWRpYS5sb2FkKCkpCisKK0VWRU5UKGxvYWRzdGFy dCkKK0VWRU5UKGR1cmF0aW9uY2hhbmdlKQorRVZFTlQobG9hZGVkZGF0YSkKK0VWRU5UKGxvYWQp CisKK1JVTihkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgncGFyZW50JykuYXBwZW5kQ2hpbGQobWVk aWEpKQorUlVOKG1lZGlhLnBsYXkoKSkKKworRVZFTlQocGxheSkKK0VWRU5UKHBsYXlpbmcpCitF TkQgT0YgVEVTVAorCkluZGV4OiBMYXlvdXRUZXN0cy9tZWRpYS9tZWRpYS1sb2FkLWV2ZW50Lmh0 bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvbWVkaWEvbWVkaWEtbG9hZC1ldmVudC5odG1s CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvbWVkaWEvbWVkaWEtbG9hZC1ldmVudC5odG1s CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDQ4IEBACis8aHRtbD4KKyAgICA8aGVhZD4KKyAgICAg ICAgPHNjcmlwdCBzcmM9dmlkZW8tdGVzdC5qcz48L3NjcmlwdD4KKworICAgICAgICA8c2NyaXB0 PgorCisgICAgICAgICAgICBmdW5jdGlvbiBsb2FkZWQoKQorICAgICAgICAgICAgeworICAgICAg ICAgICAgICAgIGNvbnNvbGVXcml0ZSgiRVZFTlQobG9hZCkiKTsKKyAgICAgICAgICAgICAgICBj b25zb2xlV3JpdGUoIiIpOworICAgICAgICAgICAgICAgIHJ1bigiZG9jdW1lbnQuZ2V0RWxlbWVu dEJ5SWQoJ3BhcmVudCcpLmFwcGVuZENoaWxkKG1lZGlhKSIpOworICAgICAgICAgICAgICAgIHJ1 bigibWVkaWEucGxheSgpIik7CisgICAgICAgICAgICAgICAgd2luZG93LnNldFRpbWVvdXQoZW5k VGVzdCwgMjUwKTsKKyAgICAgICAgICAgICAgICBjb25zb2xlV3JpdGUoIiIpOworICAgICAgICAg ICAgfQorCisgICAgICAgICAgICBmdW5jdGlvbiBzdGFydCgpCisgICAgICAgICAgICB7CisgICAg ICAgICAgICAgICAgcnVuKCJtZWRpYSA9IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoJ2F1ZGlvJyki KTsKKworICAgICAgICAgICAgICAgIG1lZGlhLm9ubG9hZCA9IGxvYWRlZDsKKworICAgICAgICAg ICAgICAgIHdhaXRGb3JFdmVudCgibG9hZHN0YXJ0Iik7CisgICAgICAgICAgICAgICAgd2FpdEZv ckV2ZW50KCJ3YWl0aW5nIik7CisgICAgICAgICAgICAgICAgd2FpdEZvckV2ZW50KCJyYXRlY2hh bmdlIik7CisgICAgICAgICAgICAgICAgd2FpdEZvckV2ZW50KCJkdXJhdGlvbmNoYW5nZSIpOwor ICAgICAgICAgICAgICAgIHdhaXRGb3JFdmVudCgicGF1c2UiKTsKKyAgICAgICAgICAgICAgICB3 YWl0Rm9yRXZlbnQoInBsYXkiKTsKKyAgICAgICAgICAgICAgICB3YWl0Rm9yRXZlbnQoInBsYXlp bmciKTsKKyAgICAgICAgICAgICAgICB3YWl0Rm9yRXZlbnQoJ2xvYWRlZGRhdGEnKTsKKworICAg ICAgICAgICAgICAgIHJ1bigibWVkaWEuc3JjID0gJ2NvbnRlbnQvdGVzdC53YXYnIik7CisgICAg ICAgICAgICAgICAgcnVuKCJtZWRpYS5sb2FkKCkiKTsKKworICAgICAgICAgICAgICAgIGNvbnNv bGVXcml0ZSgiIik7CisgICAgICAgICAgICB9CisKKyAgICAgICAgPC9zY3JpcHQ+CisgICAgPC9o ZWFkPgorCisgICAgPGJvZHkgb25sb2FkPSJzdGFydCgpIj4KKworICAgIDxwPlRlc3QgdGhhdCBt ZWRpYSBmaWxlIGlzIG5vdCByZWxvYWRlZCB3aGVuIGFuIGVsZW1lbnQgaXMgaW5zZXJ0ZWQgaW50 byB0aGUgRE9NLjwvcD4KKworICAgIDxkaXYgaWQ9InBhcmVudCI+PC9kaXY+CisgICAgCisgICAg PC9ib2R5PgorPC9odG1sPgo=