Title: [170437] trunk/Source/WebCore
Revision
170437
Author
jer.no...@apple.com
Date
2014-06-25 14:31:35 -0700 (Wed, 25 Jun 2014)

Log Message

[MSE] Update monitorSourceBuffers to match recent spec changes
https://bugs.webkit.org/show_bug.cgi?id=134305

Reviewed by Eric Carlson.

In W3C bug #24347, the text for the third clause of Source Buffer Monitoring was updated. Update our
implementation to match the new text.

* Modules/mediasource/MediaSource.cpp:
(WebCore::MediaSource::monitorSourceBuffers):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (170436 => 170437)


--- trunk/Source/WebCore/ChangeLog	2014-06-25 21:15:37 UTC (rev 170436)
+++ trunk/Source/WebCore/ChangeLog	2014-06-25 21:31:35 UTC (rev 170437)
@@ -1,3 +1,16 @@
+2014-06-25  Jer Noble  <jer.no...@apple.com>
+
+        [MSE] Update monitorSourceBuffers to match recent spec changes
+        https://bugs.webkit.org/show_bug.cgi?id=134305
+
+        Reviewed by Eric Carlson.
+
+        In W3C bug #24347, the text for the third clause of Source Buffer Monitoring was updated. Update our
+        implementation to match the new text.
+
+        * Modules/mediasource/MediaSource.cpp:
+        (WebCore::MediaSource::monitorSourceBuffers):
+
 2014-06-25  Alex Christensen  <achristen...@webkit.org>
             And pe...@outlook.com  <pe...@outlook.com>
 

Modified: trunk/Source/WebCore/Modules/mediasource/MediaSource.cpp (170436 => 170437)


--- trunk/Source/WebCore/Modules/mediasource/MediaSource.cpp	2014-06-25 21:15:37 UTC (rev 170436)
+++ trunk/Source/WebCore/Modules/mediasource/MediaSource.cpp	2014-06-25 21:31:35 UTC (rev 170437)
@@ -221,9 +221,9 @@
         return;
     }
 
-    // ↳ If buffered for at least one object in activeSourceBuffers contains a TimeRange that includes
-    // the current playback position but not enough data to ensure uninterrupted playback:
-    if (std::any_of(begin, end, [](RefPtr<SourceBuffer>& sourceBuffer) {
+    // ↳ If buffered for all objects in activeSourceBuffers contain a TimeRange that includes
+    // the current playback position and some time beyond the current playback position, then run the following steps:
+    if (std::all_of(begin, end, [](RefPtr<SourceBuffer>& sourceBuffer) {
         return sourceBuffer->hasFutureTime();
     })) {
         // 1. Set the HTMLMediaElement.readyState attribute to HAVE_FUTURE_DATA.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to