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

Description Andrew Scherkus 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!
Comment 1 Andrew Scherkus 2011-10-12 16:06:34 PDT
Created attachment 110764 [details]
Patch
Comment 2 WebKit Review Bot 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>
Comment 3 WebKit Review Bot 2011-10-13 11:50:28 PDT
All reviewed patches have been landed.  Closing bug.