From 8d0615fc9030e7fbca886804b4b93ef716a52290 Mon Sep 17 00:00:00 2001 From: mruepp Date: Wed, 29 Jun 2016 15:53:09 +0200 Subject: [PATCH] Fixed in setBufferChecker, which keeps video in pause state. Checking for currentTime + bufferInterval may fail in some browsers, cause bufferInterval is an implementation details of a browser and hence not required for the check. --- src/synchronize.js | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/src/synchronize.js b/src/synchronize.js index 3879756..aa96f71 100644 --- a/src/synchronize.js +++ b/src/synchronize.js @@ -752,17 +752,11 @@ for (i = 0; i < videoIds.length; ++i) { var bufferedTimeRange = getBufferTimeRange(videoIds[i]); if (bufferedTimeRange) { - var duration = getDuration(videoIds[i]); - var currTimePlusBuffer = getCurrentTime(videoIds[i]) + bufferInterval; + var currentTime = getCurrentTime(videoIds[i]); var buffered = false; for (var j = 0; - (j < bufferedTimeRange.length) && !buffered; ++j) { - currTimePlusBuffer = (currTimePlusBuffer >= duration) ? duration : currTimePlusBuffer; - if (isInInterval(currTimePlusBuffer, - bufferedTimeRange.start(j), - bufferedTimeRange.end(j) + bufferThreshold)) { - buffered = true; - } + (j < bufferedTimeRange.length) && !buffered; ++j) { + buffered = isInInterval(currentTime, bufferedTimeRange.start(j), bufferedTimeRange.end(j)); } allBuffered = allBuffered && buffered; isBuffering = !allBuffered;