Bug 69974 - http/tests/media/video-play-stall.html closes the connection too soon
Summary: http/tests/media/video-play-stall.html closes the connection too soon
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Andrew Scherkus
Depends on:
Reported: 2011-10-12 16:05 PDT by Andrew Scherkus
Modified: 2011-10-13 11:50 PDT (History)
2 users (show)

See Also:

Patch (1.63 KB, patch)
2011-10-12 16:06 PDT, Andrew Scherkus
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.

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]
Comment 2 WebKit Review Bot 2011-10-13 11:50:24 PDT
Comment on attachment 110764 [details]

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.