69974 2011-10-12 16:05:38 -0700 http/tests/media/video-play-stall.html closes the connection too soon 2011-10-13 11:50:28 -0700 1 1 1 Unclassified WebKit Media 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 scherkus scherkus eric.carlson webkit.review.bot oldest_to_newest 482829 0 scherkus 2011-10-12 16:05:38 -0700 I'm noticing that the test server is closing the connection sooner than the 3 second window for firing a stalled event. Chromium interprets a preemptively closed connection as an error and we stop playback before getting to fire a stalled event. http://www.whatwg.org/specs/web-apps/current-work/multipage/video.html#dom-mediaerror-media_err_network On Safari/QuickTime I noticed the following: 1st load/uncached: page stalls for a bit but then plays 3s of the video but in a loop but it doesn't fire ended events nor is loop attribute set 2nd+ load/cached: video never plays, calls to play() don't do anything but setting currentTime makes the video play in a loop again Anyway, one way of working around the early connection close is to add a stallFor parameter to make the sure that we wait long enough, but I admit it does feel like we're sweeping the issue under the rug so I'm open to suggestions! 482831 1 110764 scherkus 2011-10-12 16:06:34 -0700 Created attachment 110764 Patch 483354 2 110764 webkit.review.bot 2011-10-13 11:50:24 -0700 Comment on attachment 110764 Patch Clearing flags on attachment: 110764 Committed r97376: <http://trac.webkit.org/changeset/97376> 483355 3 webkit.review.bot 2011-10-13 11:50:28 -0700 All reviewed patches have been landed. Closing bug. 110764 2011-10-12 16:06:34 -0700 2011-10-13 11:50:24 -0700 Patch bug-69974-20111012160633.patch text/plain 1673 scherkus U3VidmVyc2lvbiBSZXZpc2lvbjogOTcyNTEKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5n ZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwppbmRleCBhYTU2NzQzNjMxYmIyMjdlM2U3Yzcz MzJlZmE0Nzk3ODhiMGMwOTE3Li42ODFkNzM0ZGY5NmE4N2NjNWYyNmM4NTA3NGEwZjliNWYwYzMy YzYyIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMv Q2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTEtMTAtMTIgIEFuZHJldyBTY2hlcmt1cyAg PHNjaGVya3VzQGNocm9taXVtLm9yZz4KKworICAgICAgICBBZGQgc3RhbGxGb3IgVVJMIHBhcmFt ZXRlciBmb3IgaHR0cC90ZXN0cy9tZWRpYS92aWRlby1wbGF5LXN0YWxsLmh0bWwgdGVzdC4KKyAg ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTY5OTc0CisKKyAg ICAgICAgV2l0aG91dCB0aGUgY2FsbCB0byBzbGVlcCgpLCB0aGUgdGVzdCBzZXJ2ZXIgd2lsbCBp c3N1ZSBhIFRDUAorICAgICAgICBjb25uZWN0aW9uIGNsb3NlIGVhcmxpZXIgdGhhbiB0aGUgMyBz ZWNvbmQgd2FpdCB0aW1lIHJlcXVpcmVkIHRvIGlzc3VlCisgICAgICAgIGEgc3RhbGxlZCBldmVu dC4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGh0 dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1zdGFsbC5odG1sOgorCiAyMDExLTEwLTEyICBQYXZl bCBQb2Rpdmlsb3YgIDxwb2Rpdmlsb3ZAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFVucmV2aWV3 ZWQsIG1hcmsgZmFzdC9jc3MvdGV4dC10cmFuc2Zvcm0tc2VsZWN0Lmh0bWwgYXMgSU1BR0UrVEVY VCBvbiB3aW4uCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9odHRwL3Rlc3RzL21lZGlhL3ZpZGVv LXBsYXktc3RhbGwuaHRtbCBiL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxh eS1zdGFsbC5odG1sCmluZGV4IGZiMWVkZjk0NTFlZGM1YzI3MDdiMjdjNTliZTFlMWZhZGI3ZDM0 ZDguLmE0ZmZiY2JlYjE2YTgwOWNiNmRhNDRkODZmNDhmNzljZTk2NDdiMzYgMTAwNjQ0Ci0tLSBh L0xheW91dFRlc3RzL2h0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1zdGFsbC5odG1sCisrKyBi L0xheW91dFRlc3RzL2h0dHAvdGVzdHMvbWVkaWEvdmlkZW8tcGxheS1zdGFsbC5odG1sCkBAIC0z Niw2ICszNiw2IEBACiAgICAgdmFyIG1lZGlhRmlsZSA9IGZpbmRNZWRpYUZpbGUoInZpZGVvIiwg ImNvbnRlbnQvdGVzdCIpOwogICAgIHZhciBtaW1lVHlwZSA9IG1pbWVUeXBlRm9yRmlsZShtZWRp YUZpbGUpOwogCi0gICAgdmlkZW8uc3JjID0gImh0dHA6Ly8xMjcuMC4wLjE6ODAwMC9tZWRpYS92 aWRlby1sb2FkLWFuZC1zdGFsbC5jZ2k/bmFtZT0uLi8uLi8uLi9tZWRpYS8iICsgbWVkaWFGaWxl ICsgIiZtaW1lVHlwZT0iICsgbWltZVR5cGUgKyAiJnN0YWxsQXQ9MTAwMDAwIjsKKyAgICB2aWRl by5zcmMgPSAiaHR0cDovLzEyNy4wLjAuMTo4MDAwL21lZGlhL3ZpZGVvLWxvYWQtYW5kLXN0YWxs LmNnaT9uYW1lPS4uLy4uLy4uL21lZGlhLyIgKyBtZWRpYUZpbGUgKyAiJm1pbWVUeXBlPSIgKyBt aW1lVHlwZSArICImc3RhbGxBdD0xMDAwMDAmc3RhbGxGb3I9NiI7CiAgICAgcnVuKCJ2aWRlby5w bGF5KCkiKTsKIDwvc2NyaXB0Pgo=