Trigger BUFFER_NUDGE_ON_STALL before currentTime update (video-dev#1795)

dailymotion · Jun 23, 2018 · 8283b34 · 8283b34
1 parent 4fccc86
commit 8283b34
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/controller/stream-controller.js b/src/controller/stream-controller.js
@@ -1532,12 +1532,12 @@ class StreamController extends TaskLoop {
       const targetTime = currentTime + nudgeRetry * config.nudgeOffset;
       logger.log(`adjust currentTime from ${currentTime} to ${targetTime}`);
       // playback stalled in buffered area ... let's nudge currentTime to try to overcome this
-      media.currentTime = targetTime;
       hls.trigger(Event.ERROR, {
         type: ErrorTypes.MEDIA_ERROR,
         details: ErrorDetails.BUFFER_NUDGE_ON_STALL,
         fatal: false
       });
+      media.currentTime = targetTime;
     } else {
       logger.error(`still stuck in high buffer @${currentTime} after ${config.nudgeMaxRetry}, raise fatal error`);
       hls.trigger(Event.ERROR, {