Improve cue removal when first cue matches start time

Author: robwalch

src/utils/texttrack-utils.ts

@@ -94,14 +94,14 @@ export function removeCuesInRange(
   }
 }
 
-// Find first cue starting after given time.
+// Find first cue starting at or after given time.
 // Modified version of binary search O(log(n)).
-function getFirstCueIndexAfterTime(
+function getFirstCueIndexFromTime(
   cues: TextTrackCueList | TextTrackCue[],
   time: number,
 ): number {
-  // If first cue starts after time, start there
-  if (time < cues[0].startTime) {
+  // If first cue starts at or after time, start there
+  if (time <= cues[0].startTime) {
     return 0;
   }
   // If the last cue ends before time there is no overlap
@@ -112,9 +112,9 @@ function getFirstCueIndexAfterTime(
 
   let left = 0;
   let right = len;
-
+  let mid;
   while (left <= right) {
-    const mid = Math.floor((right + left) / 2);
+    mid = Math.floor((right + left) / 2);
 
     if (time < cues[mid].startTime) {
       right = mid - 1;
@@ -138,7 +138,7 @@ export function getCuesInRange(
   end: number,
 ): TextTrackCue[] {
   const cuesFound: TextTrackCue[] = [];
-  const firstCueInRange = getFirstCueIndexAfterTime(cues, start);
+  const firstCueInRange = getFirstCueIndexFromTime(cues, start);
   if (firstCueInRange > -1) {
     for (let i = firstCueInRange, len = cues.length; i < len; i++) {
       const cue = cues[i];