fix: content restarts from the beginning when snapshot restores sourc…

…e after midroll in iOS

* add autoplay to force preloading after src change in iOS, fix wip

* minor changes

* edit comments

src/snapshot.js

@@ -129,6 +129,13 @@ export function restorePlayerSnapshot(player, snapshotObject) {
     if (!snapshotObject.ended) {
       player.play();
     }
+
+    // if we added autoplay to force content loading on iOS, remove it now
+    // that it has served its purpose
+    if (player.ads.shouldRemoveAutoplay_) {
+      player.autoplay(false);
+      player.ads.shouldRemoveAutoplay_ = false;
+    }
   };
 
   // determine if the video element has loaded enough of the snapshot source
@@ -197,6 +204,16 @@ export function restorePlayerSnapshot(player, snapshotObject) {
     // on ios7, fiddling with textTracks too early will cause safari to crash
     player.one('contentloadedmetadata', restoreTracks);
 
+    // adding autoplay guarantees that Safari will load the content so we can
+    // seek back to the correct time after ads
+    if (videojs.browser.IS_IOS && !player.autoplay()) {
+      player.autoplay(true);
+
+      // if we get here, the player was not originally configured to autoplay,
+      // so we should remove it after it has served its purpose
+      player.ads.shouldRemoveAutoplay_ = true;
+    }
+
     // if the src changed for ad playback, reset it
     player.src({ src: snapshotObject.currentSrc, type: snapshotObject.type });