Summary: | Do not pause movie when readyState drops below HAVE_FUTURE_DATA | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Carlson <eric.carlson> | ||||
Component: | Media | Assignee: | Eric Carlson <eric.carlson> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, eric, jer.noble, webkit.review.bot | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Eric Carlson
2010-04-22 08:30:16 PDT
Created attachment 54066 [details]
Proposed patch
Comment on attachment 54066 [details] Proposed patch > bool HTMLMediaElement::potentiallyPlaying() const > { > - return m_readyState >= HAVE_FUTURE_DATA && couldPlayIfEnoughData(); > + bool pausedToBuffer = m_readyStateMaximum >= HAVE_FUTURE_DATA && m_readyState < HAVE_FUTURE_DATA; > + return (pausedToBuffer || m_readyState >= HAVE_FUTURE_DATA) && couldPlayIfEnoughData(); I think a comment here would be useful, to explain exactly what pausedToBuffer means. trac.webkit.org/changeset/58107 http://trac.webkit.org/changeset/58107 might have broken Leopard Intel Release (Tests) Yup. Looks like this broke Tiger and Leopard test machines: http://build.webkit.org/results/Leopard%20Intel%20Debug%20(Tests)/r58107%20(13331)/results.html I am looking into it. Looks like this change has exposed a QuickTime/QTKit bug. Continuing to dig. Will need to spend some quality time in QuickTime to figure this out, so these tests added to the Leopard and Tiger skipped lists in http://trac.webkit.org/changeset/58109 |