From 09f244150c85c76994a46de862fe2c5879fd358e Mon Sep 17 00:00:00 2001 From: Ajay Ramachandran Date: Mon, 9 Mar 2020 23:00:39 -0400 Subject: [PATCH] Fixed skipping for non zero second sponsors. Also now using the playing event to fix issues with mobile YouTube skipping. --- src/content.ts | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/src/content.ts b/src/content.ts index e53e059b..37212201 100644 --- a/src/content.ts +++ b/src/content.ts @@ -449,7 +449,7 @@ function startSponsorSchedule(currentTime?: number): void { return; } - if (currentTime === undefined) currentTime = video.currentTime; + if (currentTime === undefined || currentTime === null) currentTime = video.currentTime; let skipInfo = getNextSkipIndex(currentTime); @@ -459,16 +459,19 @@ function startSponsorSchedule(currentTime?: number): void { let timeUntilSponsor = skipTime[0] - currentTime; let skippingFunction = () => { + let forcedSkipTime: number = null; + if (video.currentTime >= skipTime[0] && video.currentTime < skipTime[1]) { skipToTime(video, skipInfo.index, skipInfo.array, skipInfo.openNotice); + + if (Config.config.disableAutoSkip) { + forcedSkipTime = skipTime[0] + 0.001; + } else { + forcedSkipTime = skipTime[1]; + } } - if (Config.config.disableAutoSkip) { - startSponsorSchedule(skipTime[0] + 0.001); - } else { - startSponsorSchedule(skipTime[1]); - } - + startSponsorSchedule(forcedSkipTime); }; if (timeUntilSponsor <= 0) { @@ -500,6 +503,7 @@ function sponsorsLookup(id: string, channelIDPromise?) { switchingVideos = false; startSponsorSchedule(); }); + video.addEventListener('playing', () => startSponsorSchedule()); video.addEventListener('seeked', () => { if (!video.paused) startSponsorSchedule(); });