Bug 69974

Summary: http/tests/media/video-play-stall.html closes the connection too soon
Product: WebKit Reporter: Andrew Scherkus <scherkus>
Component: MediaAssignee: Andrew Scherkus <scherkus>
Status: RESOLVED FIXED    
Severity: Normal CC: eric.carlson, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Andrew Scherkus
Reported 2011-10-12 16:05:38 PDT
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!
Attachments
Patch (1.63 KB, patch)
2011-10-12 16:06 PDT, Andrew Scherkus 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andrew Scherkus
Comment 1 2011-10-12 16:06:34 PDT
Created attachment 110764 [details] Patch
WebKit Review Bot
Comment 2 2011-10-13 11:50:24 PDT
Comment on attachment 110764 [details] Patch Clearing flags on attachment: 110764 Committed r97376: <http://trac.webkit.org/changeset/97376>
WebKit Review Bot
Comment 3 2011-10-13 11:50:28 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.